ramitg254 commented on code in PR #5715:
URL: https://github.com/apache/hive/pull/5715#discussion_r2082451182


##########
ql/src/test/queries/clientpositive/orc_drop_column.q:
##########
@@ -0,0 +1,54 @@
+-- SORT_QUERY_RESULTS
+set hive.cli.print.header=true;
+set hive.exec.schema.evolution=false;
+
+drop table if exists tbl_orc;
+
+create external table tbl_orc(a int, b string, c int) partitioned by (d 
string) stored as orc;

Review Comment:
   i think it is due to removal of
   ```
   if (AcidUtils.isTransactionalTable(table)) {
         setAcidDdlDesc(desc);
       }
   ```
   as ACID tables are not supported for dropping columns case of replace 
columns and other things like changing order and addition of columns through it 
are supported so need to revert the changes of  
`AlterTableReplaceColumnsAnalyzer.java`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to