[GitHub] [pinot] swaminathanmanish commented on pull request #10856: Add the support for filling the default header if the header is missing

["swaminathanmanish (via GitHub)" <gi...@apache.org>]

swaminathanmanish commented on PR #10856:
URL: https://github.com/apache/pinot/pull/10856#issuecomment-1579337132

   Can we also document user experience when this feature is enabled ? 
   1. What happens if proper headers are there already ? Here the algorithm should not get this wrong and override the proper headers. How do we safeguard against this?
   2. What happens when there are no headers and we assign default headers
   3. What happens when there are no headers and we fail to detect and assign default
   
   


-- 
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-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org