You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/08/04 16:39:21 UTC

[GitHub] [camel] jam01 commented on pull request #8023: CAMEL-18292: Addresses ujson.IncompleteParseException: exhausted input error when …

jam01 commented on PR #8023:
URL: https://github.com/apache/camel/pull/8023#issuecomment-1205506130

   DataSonnet does accept `null`, you can try this:
   ```
   System.out.println(new Mapper("payload").transform(new DefaultDocument<>(null)).getContent());
   // the output is the String "null"
   ```
   
   Further, changing a `null` to empty string is a potentially breaking change for anyone that's checking for `null` payload.
   
   I propose we roll this change back and figure out the root cause of the ujson exception. Perhaps checking for null body and if so setting the `DefaulDocument.NULL_INSTANCE`
   
   @javaduke @davsclaus 


-- 
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@camel.apache.org

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