npawar opened a new pull request #5126: Change console loggers to non-async for pinot-tools URL: https://github.com/apache/incubator-pinot/pull/5126 Someone reached out to us on the slack channel that they were trying out quickstart, but cannot see any logs when running the pinot-admin commands. Specifically, the issues were for AddTable and LaunchDataIngestionJob. I was able to reproduce the issue. I setup the quick-start-batch cluster, and then `AddTable` command with missing table config file. ``` # checkout and build, and then bin/quick-start-batch.sh bin/pinot-admin.sh AddTable -tableConfigFile /tmp/pinot-quick-start/transcript-table.json -exec ``` There's just 1 line of log, and the program exits ``` 2020/03/06 17:08:19.205 INFO [AddTableCommand] [main] Executing command: AddTable -tableConfigFile /tmp/pinot-quick-start/transcript-table-offline.json -schemaFile /tmp/pinot-quick-start/transcript-schema.json -controllerHost xxxx -controllerPort 9000 -exec ``` Possible suspect is the AsyncLogger. On changing `s/AsyncLogger/Logger`, this works for me. We should be fine changing AsyncLogger to Logger for `pinot-tools` module. Other suggestions welcome.
---------------------------------------------------------------- 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]
