npawar commented on issue #3877: Pinot controller side change to enhance LLC segment metadata upload. URL: https://github.com/apache/incubator-pinot/pull/3877#issuecomment-467681135 > > Also, can you modify the existing SegmentCommitEnd to take on an optional 2-part form? [#3748 (comment)](https://github.com/apache/incubator-pinot/pull/3748#issuecomment-461229243) > > To clarify. The current SegmentCommitEnd() is a GET method. Can a GET method take a form? It looks to me abuse of GET. If we change to POST, would that be a back compatibility issue? Ah yes, existing method is GET. In that case I think what you have is the best way
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
