Github user prashanth-vasudev commented on a diff in the pull request:

    https://github.com/apache/trafodion/pull/1532#discussion_r193887315
  
    --- Diff: core/sql/src/main/java/org/trafodion/sql/HBaseClient.java ---
    @@ -538,7 +564,21 @@ public boolean createk(String tblName, Object[] 
tableOptions,
                          admin.createTable(desc);
                       }
                    }
    -            admin.close();
    +
    +            if(setDescRet!= null)
    +            {
    +              if(setDescRet.storagePolicyChanged())
    +              {
    +                 Object tableOptionsStoragePolicy[] = new 
Object[HBASE_HDFS_STORAGE_POLICY+1];
    +                 for(int i=0; i<HBASE_HDFS_STORAGE_POLICY; i++)
    +                   tableOptionsStoragePolicy[i]="";
    +                 
tableOptionsStoragePolicy[HBASE_HDFS_STORAGE_POLICY]=(String)setDescRet.storagePolicy_
 ;
    +                 tableOptionsStoragePolicy[HBASE_NAME]=(String)tblName;
    +                 alter(tblName,tableOptionsStoragePolicy,transID);
    --- End diff --
    
    Transactional Alter is not supported yet. This is due to additional changes 
required to support transactional rollback of a table.  Can you make 
coprocessor call outside of transactional alter?


---

Reply via email to