[GitHub] [pinot] amrishlal commented on pull request #7873: [DO NOT REVIEW] Fix JSON path expressions in SQL queries.

[GitBox <gi...@apache.org>]

amrishlal commented on pull request #7873:
URL: https://github.com/apache/pinot/pull/7873#issuecomment-987214407


   > This isn't a review comment but can you let me know if you intend to make any changes on the server please?
   
   @richardstartin Only BaseBrokerRequestHandler.java is being changed. Along with that I will add an integration test that will detect future breakages.
   
   @mayankshriv @kishoreg This isn't a new feature, this is a bug fix. The JSON naming convention that is described in 
   
   I will add a description with more detailed information. Please note that this is not a new feature and there is no change to any existing functionality. The PR is fixing a regression which crept in due to a prior commit (https://github.com/apache/pinot/pull/7590).


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