pnoltes commented on PR #842:
URL: https://github.com/apache/celix/pull/842#issuecomment-5285084472

   Very interesting. I did a quick look and this looks like a nice addition. I 
do need some time to review this correctly, but will start planning that. 
   
   BTW, I was also working on JSONPath and extending array list and properties 
so that a more complete JSON support is possible (NULL value, mixed array 
support, nested properties). When this has been reviewed and merged, I will see 
if the NULL value, mixed array support and nested properties is still needed. 
   
   In this branch I was working on, filter also has some support for JSONPath 
usage; But I was not yet sure too keep that (possible high performance impact 
on filter evaluation and therefore service lookup).  
   
   
https://github.com/apache/celix/compare/master...pnoltes:celix:feature/json-props-update-plan


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to