kautshukla opened a new issue #7773:
URL: https://github.com/apache/pinot/issues/7773


   
https://github.com/apache/pinot/blob/d1606cd0f3877c1a8ec08255d003ef104a94c500/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotDriver.java#L71
   
   Prerequisite: 
https://github.com/apache/pinot/blob/d1606cd0f3877c1a8ec08255d003ef104a94c500/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotDriver.java#L60
 : headers map is not empty {JDBC with headers for Auth enabled client}
   
   As of now:
                 return new PinotConnection(controllerUrl, 
pinotClientTransport, tenant);
   
   It should be:
                if(!headers.isEmpty)
                    return new PinotConnection(controllerUrl, 
pinotClientTransport, tenant, new PinotControllerTransport(headers));
                return new PinotConnection(controllerUrl, pinotClientTransport, 
tenant);


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to