Hello Fredy Wijaya, Philip Zeyliger, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12079

to look at the new patch set (#4).

Change subject: IMPALA-7968, Part 1: JSON serialization framework
......................................................................

IMPALA-7968, Part 1: JSON serialization framework

Provides a set of interfaces and classes to serialize the analyzer AST
to JSON. This part provides the serialization framework itself, later
parts will provide the serialization code and test framework.

The framework is modular with three implementations:

* Jackson streaming API to produce standard JSON for export to external
  systems.
* Bespoke JSON serializer optimized to produce compact, repeatable
  output for testing. This form may be adapted to produce YAML, which is
  more compact than JSON in order to reduce test file size.
* JSONSimple (already an Impala dependency) to create JSON objects. Not
  super useful, but shows how to use other object-based JSON frameworks
  if needed.

This framework itself is independent of the AST and can also be used to
serialize the physical plan as well.

Testing: test cases exercise the mechanisms. The code is an island in
the patch; no other code uses it yet.

Change-Id: Ie3101c2708bf6cf4bec61af83a5db9b6d70ddd9c
---
A 
fe/src/main/java/org/apache/impala/common/serialize/AbstractTreeSerializer.java
A fe/src/main/java/org/apache/impala/common/serialize/ArraySerializer.java
A fe/src/main/java/org/apache/impala/common/serialize/JacksonTreeSerializer.java
A fe/src/main/java/org/apache/impala/common/serialize/JsonSerializable.java
A fe/src/main/java/org/apache/impala/common/serialize/JsonSerializer.java
A fe/src/main/java/org/apache/impala/common/serialize/JsonTreeFormatter.java
A fe/src/main/java/org/apache/impala/common/serialize/ObjectSerializer.java
A fe/src/main/java/org/apache/impala/common/serialize/SerializationError.java
A fe/src/main/java/org/apache/impala/common/serialize/ToJsonOptions.java
A fe/src/main/java/org/apache/impala/common/serialize/TreeSerializer.java
A 
fe/src/test/java/org/apache/impala/common/serialize/JacksonTreeSerializerTest.java
A fe/src/test/java/org/apache/impala/common/serialize/TreeFormatterTest.java
12 files changed, 1,944 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/79/12079/4
--
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: newpatchset
Gerrit-Change-Id: Ie3101c2708bf6cf4bec61af83a5db9b6d70ddd9c
Gerrit-Change-Number: 12079
Gerrit-PatchSet: 4
Gerrit-Owner: Paul Rogers <par0...@yahoo.com>
Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>

Reply via email to