tibrewalpratik17 opened a new pull request, #10293:
URL: https://github.com/apache/pinot/pull/10293

   In the `AbstractBaseAdminCommand` class, while calling 
`readInputStream(conn.getErrorStream())` leads to NullPointerException if the 
connection is failed to establish while sending request. According to docs:
   
   > This method will not cause a connection to be initiated. If the connection 
was not connected, or if the server did not have an error while connecting or 
if the server had an error but no error data was sent, this method will return 
null. 
   
   This patch checks if `getErrorStream() != null` and throws exception 
accordingly.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to