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


##########
ql/src/java/org/apache/hadoop/hive/ql/ddl/table/create/CreateTableOperation.java:
##########
@@ -161,7 +161,9 @@ private void createTableNonReplaceMode(Table tbl) throws 
HiveException {
     if (desc.isCTAS()) {
       Table createdTable = context.getDb().getTable(tbl.getDbName(), 
tbl.getTableName());
       DataContainer dc = new DataContainer(createdTable.getTTable());
-      
context.getQueryState().getLineageState().setLineage(createdTable.getPath(), 
dc, createdTable.getCols());
+      context.getQueryState().getLineageState().setLineage(
+          createdTable.getPath(), dc, 
createdTable.hasNonNativePartitionSupport() ?

Review Comment:
   I went for a wrapper as there are other places which should have similar 
kind of behaviour for lineage but didn't got caught in ci, so it looked better 
than to have ternary at those places as well
   please check 
https://github.com/apache/hive/commit/1cfb1c7483ccda9ab712eba74e5b13fc154c4f5c



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to