[GitHub] [pinot] richardstartin commented on pull request #7820: push JSON Path evaluation down to storage layer

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

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


   @kishoreg Specifying in Javadoc that these are unstable APIs is an excellent compromise. I think it should be possible to evolve towards what you want. I must admit that I don't see it right now, but I'm trying hard to find a way both to implement the suggestion and am thinking about the benefits. I want to make it very easy to implement storage plugins (as an idea incubation mechanism) without needing community consensus on the shape of SPIs like `ForwardIndexReader` which will affect a lot of stakeholders. A delegation chain of `TransformEvaluator`s, marked as an unstable interface, feels like a good way to iterate without stalling for consensus or imposing churn on stakeholders.


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