sankarh commented on a change in pull request #588: HIVE-20968 : Support conversion of managed to external where location set was not owned by hive URL: https://github.com/apache/hive/pull/588#discussion_r273405034
########## File path: ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java ########## @@ -203,15 +203,15 @@ private void parsePartitionSpec(ASTNode tableNode, LinkedHashMap<String, String> } private static void upgradeTableDesc(org.apache.hadoop.hive.metastore.api.Table tableObj, MetaData rv, - EximUtil.SemanticAnalyzerWrapperContext x) + EximUtil.SemanticAnalyzerWrapperContext x, Boolean isPathOwnedByHive) Review comment: Shall use primitive boolean type instead of class Boolean. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
