[ https://issues.apache.org/jira/browse/HBASE-9283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13752912#comment-13752912 ]
Hudson commented on HBASE-9283: ------------------------------- SUCCESS: Integrated in hbase-0.95 #500 (See [https://builds.apache.org/job/hbase-0.95/500/]) HBASE-9283 Struct and StructIterator should properly handle trailing nulls (stack: rev 1518319) * /hbase/branches/0.95/hbase-common/src/main/java/org/apache/hadoop/hbase/types/Struct.java * /hbase/branches/0.95/hbase-common/src/main/java/org/apache/hadoop/hbase/types/StructIterator.java * /hbase/branches/0.95/hbase-common/src/test/java/org/apache/hadoop/hbase/types/TestStruct.java * /hbase/branches/0.95/hbase-common/src/test/java/org/apache/hadoop/hbase/types/TestStructNullExtension.java > Struct and StructIterator should properly handle trailing nulls > --------------------------------------------------------------- > > Key: HBASE-9283 > URL: https://issues.apache.org/jira/browse/HBASE-9283 > Project: HBase > Issue Type: Bug > Affects Versions: 0.95.2 > Reporter: Nick Dimiduk > Assignee: Nick Dimiduk > Fix For: 0.98.0, 0.96.0 > > Attachments: 0001-HBASE-9283-Struct-trailing-null-handling.patch, > 0001-HBASE-9283-Struct-trailing-null-handling.patch, > 0001-HBASE-9283-Struct-trailing-null-handling.patch > > > For a composite row key, Phoenix strips off trailing null columns values in > the row key. The reason this is important is that then new nullable row key > columns can be added to a schema without requiring any data upgrade to > existing rows. Otherwise, adding new row key columns to the end of a schema > becomes extremely cumbersome, as you'd need to delete all existing rows and > add them back with a row key that includes a null value. > Rather than Phoenix needing to modify the iteration code everywhere (as > [~ndimiduk] outlined here: > https://issues.apache.org/jira/browse/HBASE-8693?focusedCommentId=13744499&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13744499), > it'd be better if StructIterator handled this out-of-the-box. Otherwise, if > Phoenix has to specialize this, we'd lose the interop piece which is the > justification for switching our type system to this new one in the first > place. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira