> On April 28, 2018, 1:52 a.m., Eugene Koifman wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/repl/dump/io/FileOperations.java > > Lines 117 (patched) > > <https://reviews.apache.org/r/66720/diff/3/?file=2010589#file2010589line122> > > > > this should include getOriginalFiles() check if table was converted to > > MM but not yet compacted (I assume the patch to make this conversion > > metadata-only operation is still somewhere in flight)
Will be fixed in the patch that adds original files support that will likely be committed after this patch. Or in this patch if that one goes first. - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66720/#review202100 ----------------------------------------------------------- On April 23, 2018, 9:18 p.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/66720/ > ----------------------------------------------------------- > > (Updated April 23, 2018, 9:18 p.m.) > > > Review request for hive and Eugene Koifman. > > > Repository: hive-git > > > Description > ------- > > . > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/CopyTask.java ce683c8a8d > ql/src/java/org/apache/hadoop/hive/ql/exec/ExportTask.java aba65918f8 > ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 6395c31ec7 > ql/src/java/org/apache/hadoop/hive/ql/exec/repl/ReplDumpTask.java > ce0757cba2 > ql/src/java/org/apache/hadoop/hive/ql/parse/ExportSemanticAnalyzer.java > d3c62a2775 > ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java > b850ddc9d0 > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzerFactory.java > 820046388a > ql/src/java/org/apache/hadoop/hive/ql/parse/repl/dump/PartitionExport.java > 5844f3d97f > ql/src/java/org/apache/hadoop/hive/ql/parse/repl/dump/TableExport.java > abb2e8874b > > ql/src/java/org/apache/hadoop/hive/ql/parse/repl/dump/io/FileOperations.java > 866d3513b1 > ql/src/java/org/apache/hadoop/hive/ql/plan/CopyWork.java c0e4a43d9c > ql/src/java/org/apache/hadoop/hive/ql/plan/ExportWork.java 72ce79836c > ql/src/test/org/apache/hadoop/hive/ql/TestTxnCommands.java 12d57c6feb > ql/src/test/org/apache/hadoop/hive/ql/TestTxnExIm.java 0e53697be2 > ql/src/test/org/apache/hadoop/hive/ql/TxnCommandsBaseForTests.java > a2adb966fe > ql/src/test/queries/clientpositive/mm_exim.q c47342bd23 > ql/src/test/results/clientpositive/llap/mm_exim.q.out 1f40754373 > > > Diff: https://reviews.apache.org/r/66720/diff/3/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
