You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/01/31 21:22:05 UTC

[GitHub] [pinot] richardstartin commented on pull request #7819: Upgrade to JsonPath 2.7.0

richardstartin commented on pull request #7819:
URL: https://github.com/apache/pinot/pull/7819#issuecomment-1026223193


   > Thanks for contributing the fix to the json-path library!
   > In `JsonFunctions.java` I can still see some method throwing `JsonProcessingException`. Should it be eliminated completely?
   
   No, the values may not be valid json, or it may not be possible to stringify. The exceptions not thrown now were thrown  because a jsonpath didn’t match the document.


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