You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pinot.apache.org by GitBox <gi...@apache.org> on 2019/01/24 07:02:46 UTC
[GitHub] sunithabeeram opened a new pull request #3740: Fix
controller/server/broker admin console
sunithabeeram opened a new pull request #3740: Fix controller/server/broker admin console
URL: https://github.com/apache/incubator-pinot/pull/3740
Recent upgrade to jersey-2.28-RC4 left the pinot admin consoles broken, with errors similar to: "Resource identified by path '/query', does not exist"
Traced this down to how the mappings for the different paths are setup - the newer grizzly handlers expect a trailing "/".
Changes include:
- Fix the mappings in controller/broker/server to include the trailing slash.
- Added integration test to exercise the admin paths. Verified that the tests fail without the fix.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pinot.apache.org
For additional commands, e-mail: dev-help@pinot.apache.org