[ https://issues.apache.org/jira/browse/HADOOP-2296?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12546952 ]
Hadoop QA commented on HADOOP-2296: ----------------------------------- -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12370378/2296.patch against trunk revision r599586. @author +1. The patch does not contain any @author tags. javadoc +1. The javadoc tool did not generate any warning messages. javac +1. The applied patch does not generate any new compiler warnings. findbugs +1. The patch does not introduce any new Findbugs warnings. core tests +1. The patch passed core unit tests. contrib tests -1. The patch failed contrib unit tests. Test results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1210/testReport/ Findbugs warnings: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1210/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1210/artifact/trunk/build/test/checkstyle-errors.html Console output: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1210/console This message is automatically generated. > hbase shell: phantom columns show up from select command > -------------------------------------------------------- > > Key: HADOOP-2296 > URL: https://issues.apache.org/jira/browse/HADOOP-2296 > Project: Hadoop > Issue Type: Bug > Components: contrib/hbase > Affects Versions: 0.15.0 > Reporter: Michael Bieniosek > Attachments: 2296.patch > > > {code} > Hbase> select * from hbase_test where row='2'; > +--------------------------------------+--------------------------------------+ > | Column | Cell > | > +--------------------------------------+--------------------------------------+ > | test:a | a > | > +--------------------------------------+--------------------------------------+ > | test:c | c > | > +--------------------------------------+--------------------------------------+ > 2 row(s) in set (0.00 sec) > Hbase> select * from hbase_test where row='1'; > +--------------------------------------+--------------------------------------+ > | Column | Cell > | > +--------------------------------------+--------------------------------------+ > | test:a | a > | > +--------------------------------------+--------------------------------------+ > | test:b | b > | > +--------------------------------------+--------------------------------------+ > 2 row(s) in set (0.00 sec) > Hbase> select * from hbase_test; > +-------------------------+-------------------------+-------------------------+ > | Row | Column | Cell > | > +-------------------------+-------------------------+-------------------------+ > | 1 | test:a | a > | > +-------------------------+-------------------------+-------------------------+ > | 1 | test:b | b > | > +-------------------------+-------------------------+-------------------------+ > | 2 | test:a | a > | > +-------------------------+-------------------------+-------------------------+ > | 2 | test:b | b > | > +-------------------------+-------------------------+-------------------------+ > | 2 | test:c | c > | > +-------------------------+-------------------------+-------------------------+ > 5 row(s) in set (0.14 sec) > {code} > Note the phantom value for test:b in row 2. > I looked at the code, and it looks like SelectCommand.scanPrint incorrectly > fails to call results.clear() every time it calls scan.next(). > However, I also think that the HScannerInterface.next(HStoreKey key, > SortedMap<Text,byte[]> results) is confusing, since it requires the user to > call results.clear() and key.clear() before calling next each time. Since > the Iterable interface that provides the zero-arg next has been added, I > suggest that it might be worthwhile to deprecate the two-arg next. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.