You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "Jackie-Jiang (via GitHub)" <gi...@apache.org> on 2023/06/02 22:24:37 UTC

[GitHub] [pinot] Jackie-Jiang commented on issue #10832: option(skipUpsert=true) is not case inSensitive

Jackie-Jiang commented on issue #10832:
URL: https://github.com/apache/pinot/issues/10832#issuecomment-1574383820

   All query options are case sensitive now, and pinot doesn't throw error when it cannot recognize an option. One solution would be to canonicalize the option keys, and check if the key is within an allowed key list. The problem with this approach is that it makes it hard to extend the option keys because user might want to put some options not understood by pinot, but used in their custom code plugged in


-- 
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