----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72532/ -----------------------------------------------------------
(Updated June 8, 2020, 10:58 a.m.) Review request for hive, Karen Coppage, Marta Kuczora, and Peter Vary. Repository: hive-git Description ------- since HIVE-21225 there are two redundant implementation of the AcidUtils.getAcidState. The previous implementation (without the recursive listing) can be removed. Also the performance can be improved, by removing unnecessary fileStatus calls. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java 635ed3149c ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java ca234cfb37 ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java 1059cb227f ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcRawRecordMerger.java 16c915959c ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java 598220b0c4 ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Cleaner.java 2a15913f9f ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java 4e5d5b003b ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Initiator.java 7913295380 ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/MinorQueryCompactor.java d83a50f555 ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/MmMajorQueryCompactor.java 5e11d8d2d8 ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/MmMinorQueryCompactor.java 1bdec7df2d ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/Worker.java 75941b3f33 ql/src/test/org/apache/hadoop/hive/ql/TestTxnCommands2.java 337f469d1a ql/src/test/org/apache/hadoop/hive/ql/io/TestAcidUtils.java f351f04b08 ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java e4440e9136 ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestOrcRawRecordMerger.java f63c40a7b5 streaming/src/test/org/apache/hive/streaming/TestStreaming.java 3a3b267927 Diff: https://reviews.apache.org/r/72532/diff/3/ Changes: https://reviews.apache.org/r/72532/diff/2-3/ Testing ------- Thanks, Peter Varga
