[GitHub] [geode] jdeppe-pivotal commented on pull request #5858: GEODE-8795: Lucene queries should utilize post-processing if enabled

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

jdeppe-pivotal commented on pull request #5858:
URL: https://github.com/apache/geode/pull/5858#issuecomment-747025574


   Almost all the work is still from @nabarunnag. I had to make a different change to bring the principal into the `FunctionContext`.
   
   One difference is that the original fix was not passing a `PDXInstance` (if PDX was in use) to the post-processor but was instead deserializing to the domain object first. If we just pass whatever is retrieved (PDX or otherwise) the post processor can then make a decision how to handle it. Otherwise the user needs to ensure that their domain classes are available on the server.


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

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