cbalci commented on PR #11951: URL: https://github.com/apache/pinot/pull/11951#issuecomment-1804346237
Thanks @Jackie-Jiang ! >Please take a look at the test failure. Fixed the test case to reflect the updated URL size limit. Though, I'm not sure if this test serves a meaningful purpose. I can get rid of the case if you agree. >Is the query sent by HTTP header? Why are we setting such a large HTTP header size? It is not the actual HTTP header which causes the problem but "URL + query string" size. URL + query string is counted towards the maxHttpHeaderSize and URL's such as `/debug/routingTable/sql?query=<SQL>` can fail. -- 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]
