Abhishek Chennaka has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23481 )

Change subject: KUDU-1261 [Java] Implement serdes of Array Type column
......................................................................

KUDU-1261 [Java] Implement serdes of Array Type column

Added a new directory kudu-flatbuffers, similar to the kudu-proto structure,
to hold all FlatBuffers-related build configuration. Generated Java classes
from the existing FlatBuffers schema (array1d.fbs) to represent 1-D scalar 
arrays.

Implemented Array1dSerdes for serialization and deserialization of all scalar
types defined in array1d.fbs. Added corresponding test cases for each type to
validate the serdes functionality.

The generated flatbuffer classes are shaded into the Kudu client jar.
Updated verify_jars.pl script accordingly.

checkstyle and spotbugs are configured not to run on the auto-generated classes
in kudu-flatbuffers.

Change-Id: Ie1ad9f65fe94c8662ed0e0834ce849e078fc72d2
Reviewed-on: http://gerrit.cloudera.org:8080/23481
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Abhishek Chennaka <[email protected]>
---
M build-support/verify_jars.pl
M java/config/spotbugs/excludeFilter.xml
M java/gradle/dependencies.gradle
M java/gradle/quality.gradle
M java/gradle/shadow.gradle
M java/kudu-client/build.gradle
A java/kudu-client/src/main/java/org/apache/kudu/client/Array1dSerdes.java
A java/kudu-client/src/test/java/org/apache/kudu/client/TestArraySerdes.java
A java/kudu-flatbuffers/build.gradle
M java/settings.gradle
10 files changed, 902 insertions(+), 3 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Abhishek Chennaka: Verified

--
To view, visit http://gerrit.cloudera.org:8080/23481
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1ad9f65fe94c8662ed0e0834ce849e078fc72d2
Gerrit-Change-Number: 23481
Gerrit-PatchSet: 15
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to