Abhishek Chennaka has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23564 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/23564 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Abhishek Chennaka <[email protected]> --- 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, 73 insertions(+), 36 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Abhishek Chennaka: Verified -- 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: merged Gerrit-Change-Id: I8cdd3ec034c3118b476e7d0da264819b1d4c2cd8 Gerrit-Change-Number: 23564 Gerrit-PatchSet: 7 Gerrit-Owner: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
