----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66514/ -----------------------------------------------------------
Review request for hive, Eugene Koifman and Sergey Shelukhin. Repository: hive-git Description ------- Replace usage of SessionState.getTxnMgr() from several places, by doing some refactoring to make the TxnManager available in fields passed in during construction/initialization: - SemanticAnalyzer.genFileSinkPlan() - ReplicationSemanticAnalyzer.analyzeReplLoad() - LoadSemanticAnalyzer.analyzeExternal() - ImportSemanticAnalyzer.prepareImport() - DDLSemanticAnalyzer.handleTransactionalTable() Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java fb1efe01dc ql/src/java/org/apache/hadoop/hive/ql/exec/Task.java a8d851fd81 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/ReplLoadTask.java 6b333d7184 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/LoadConstraint.java 60c85f58e5 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/LoadFunction.java bc7d0ad0b9 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/table/LoadPartitions.java 06adc64727 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/table/LoadTable.java 1395027159 ql/src/java/org/apache/hadoop/hive/ql/exec/repl/bootstrap/load/util/Context.java bb51f36a25 ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 9e66422904 ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 8b639f7922 ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java e49089b91e ql/src/java/org/apache/hadoop/hive/ql/parse/ReplicationSemanticAnalyzer.java 79b2e48ee2 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java ff0a2e6a1b ql/src/java/org/apache/hadoop/hive/ql/parse/repl/load/message/MessageHandler.java 3ccd639d62 ql/src/java/org/apache/hadoop/hive/ql/parse/repl/load/message/TableHandler.java 4cd75d8128 Diff: https://reviews.apache.org/r/66514/diff/1/ Testing ------- Thanks, Jason Dere
