navina opened a new pull request, #10216: URL: https://github.com/apache/pinot/pull/10216
* When testing server upload to deepstore, we noticed that segment store uri config is not mandatory for the server instance. The upload from server to deep store was failing due to this missing configuration. Moreover, the server logs did not contain any information about why the commit failed and the partition would just hang on the server. This PR: * Improves on the error logging * If segment store uri is missing, we will default to using segment upload to controller. * Added some quick fixes to the `StartController` and `StartServer` commands to accept config overrides via command line Labels: `bugfix` -- 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
