Philip Zeyliger has posted comments on this change. ( http://gerrit.cloudera.org:8080/12079 )
Change subject: IMPALA-7968, Part 1: JSON serialization framework ...................................................................... Patch Set 4: > > To what extent is this the Jackson streaming API? > > The Jackson Streaming API is low level, while the serialization API > provided here operates at a higher level, making assumptions about > the kinds of data we want to serialize. > > Jackson is very good for producing standard JSON. > > To show the relationship, went ahead and implemented a serializer > based on Jackson. The code is actually longer than the bespoke > version, but the Jackson version produces standard JSON we can > export to other tools. Tell me more about what you're trying to do? If we're going to call it JSON, it should probably be parseable with most JSON parsers. Various Hadoop ecosystem things use plenty of Jackson and class annotations and what-not. It's not perfect, but it seems pretty standard. -- To view, visit http://gerrit.cloudera.org:8080/12079 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie3101c2708bf6cf4bec61af83a5db9b6d70ddd9c Gerrit-Change-Number: 12079 Gerrit-PatchSet: 4 Gerrit-Owner: Paul Rogers <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Paul Rogers <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Comment-Date: Sat, 15 Dec 2018 04:16:08 +0000 Gerrit-HasComments: No
