Hello Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/23564
to look at the new patch set (#4).
Change subject: KUDU-1261 [Java] Fix ColumnSchemaBuilder copy constructor
......................................................................
KUDU-1261 [Java] Fix ColumnSchemaBuilder copy constructor
Fix ColumnSchemaBuilder(ColumnSchema) to retain array type info when
copying NESTED columns. The builder now restores the element type and
marks isArray=true, preventing loss of the nested descriptor on build().
Added ColumnSchemaTest.testCloneArrayColumn() to verify array columns
can be cloned and rebuilt without exceptions or metadata loss.
Replaced Assert.assert* with the static import style assert* for
readability.
Change-Id: I8cdd3ec034c3118b476e7d0da264819b1d4c2cd8
---
M java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java
M java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java
2 files changed, 79 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/64/23564/4
--
To view, visit http://gerrit.cloudera.org:8080/23564
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8cdd3ec034c3118b476e7d0da264819b1d4c2cd8
Gerrit-Change-Number: 23564
Gerrit-PatchSet: 4
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
