Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/973#discussion_r143753438
  
    --- Diff: 
contrib/storage-hbase/src/main/java/org/apache/drill/exec/store/hbase/HBaseGroupScan.java
 ---
    @@ -143,9 +143,9 @@ private HBaseGroupScan(HBaseGroupScan that) {
     
       @Override
       public GroupScan clone(List<SchemaPath> columns) {
    +    HBaseUtils.verifyColumns(columns, hTableDesc);
    --- End diff --
    
    We should replace null columns with all_columns before verification or we 
will get exception. You have changed initial order of statements.


---

Reply via email to