xueyumusic opened a new issue #4383: [MINOR] maybe a log bug of 
PinotAdministrator
URL: https://github.com/apache/incubator-pinot/issues/4383
 
 
   When using `pinot-admin.sh CreateSegment` if `overwrite` option is not 
specified but the output dir existed,  it did not have any related prompt on 
console.
   It should [log error in 
code](https://github.com/apache/incubator-pinot/blob/master/pinot-tools/src/main/java/org/apache/pinot/tools/admin/PinotAdministrator.java#L111),
 but looks print nothing. When using `Logger` instead of `AsyncLogger` will be 
ok. 
   A way to fix it is adding `LogManager.shutdown()` in main or 
try-catch-finally, is it a proper way?
   Thanks
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to