Re: [PR] Issue apache#2362 Evaluate JsonPath functions in the JSON Input transform (hop)

2026-09-25 Thread via GitHub


hansva merged PR #8198:
URL: https://github.com/apache/hop/pull/8198


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



Re: [PR] Issue apache#2362 Evaluate JsonPath functions in the JSON Input transform (hop)

2026-09-01 Thread via GitHub


sramazzina commented on PR #8198:
URL: https://github.com/apache/hop/pull/8198#issuecomment-5490147976

   Oh yes I forget I'll do that sorry my bad


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



[PR] Issue apache#2362 Evaluate JsonPath functions in the JSON Input transform (hop)

2026-08-31 Thread via GitHub


sramazzina opened a new pull request, #8198:
URL: https://github.com/apache/hop/pull/8198

 - JsonPath refuses to evaluate a function path (length(), sum(), keys(), 
...)
   while ALWAYS_RETURN_LIST is set and, with SUPPRESS_EXCEPTIONS, returns 
null
   instead; such a path is now re-evaluated without that option and its 
scalar
   result exposed as a single row.
 - The null returned in that case reached normalizeJsonPathResult and 
caused a
   NullPointerException while building the error message; the case is now
   handled before it gets there.
 - Document JsonPath functions in the JSON Input transform page, including 
the
   single-row constraint.
   
   Fixes #2362 
   
   
   Thank you for your contribution! Follow this checklist to help us 
incorporate your contribution quickly and easily:
   - [ ] Run `mvn clean install apache-rat:check` to make sure basic checks 
pass. A more thorough check will be performed on your pull request 
automatically.
   - [ ] If you have a group of commits related to the same change, please 
squash your commits into one and force push your branch using `git rebase -i`.
   - [ ] Mention the appropriate issue in your description (for example: 
`addresses #123`), if applicable.
   
   To make clear that you license your contribution under the [Apache License 
Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   you have to acknowledge this by using the following check-box.
   
   - [ ] I hereby declare this contribution to be licensed under the [Apache 
License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   - [ ] In any other case, please file an [Apache Individual Contributor 
License Agreement](https://www.apache.org/licenses/icla.pdf).
   


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



Re: [PR] Issue apache#2362 Evaluate JsonPath functions in the JSON Input transform (hop)

2026-08-31 Thread via GitHub


hansva commented on PR #8198:
URL: https://github.com/apache/hop/pull/8198#issuecomment-5481848450

   Add some integration tests too please 


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