[ https://issues.apache.org/jira/browse/HIVE-25521?focusedWorklogId=652977&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-652977 ]
ASF GitHub Bot logged work on HIVE-25521: ----------------------------------------- Author: ASF GitHub Bot Created on: 20/Sep/21 13:54 Start Date: 20/Sep/21 13:54 Worklog Time Spent: 10m Work Description: pgaref commented on a change in pull request #2639: URL: https://github.com/apache/hive/pull/2639#discussion_r712189151 ########## File path: ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestOrcFileStripeMergeRecordReader.java ########## @@ -63,8 +63,20 @@ public void testSplitStartsWithOffset() throws IOException { FileSplit split = new FileSplit(tmpPath, offset, length, (String[])null); OrcFileStripeMergeRecordReader reader = new OrcFileStripeMergeRecordReader(conf, split); reader.next(key, value); + // since offset is non-zero this file will not be processed. + Assert.assertNull(key.getInputPath()); + split = new FileSplit(tmpPath, 0, length, (String[]) null); Review comment: Offset is actually Zero here -- what am I missing? ########## File path: ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcFileStripeMergeRecordReader.java ########## @@ -80,7 +80,7 @@ public boolean next(OrcFileKeyWrapper key, OrcFileValueWrapper value) throws IOE } protected boolean nextStripe(OrcFileKeyWrapper keyWrapper, OrcFileValueWrapper valueWrapper) - throws IOException { + throws IOException { // missing stripe stats (old format). If numRows is 0 then its an empty file and no statistics // is present. We have to differentiate no stats (empty file) vs missing stats (old format). if ((stripeStatistics == null || stripeStatistics.isEmpty()) && reader.getNumberOfRows() > 0) { Review comment: would it make sense to have the ```start > 0``` check here instead? -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 652977) Remaining Estimate: 0h Time Spent: 10m > Data corruption when concatenating files with different compressions in same > table/partition > -------------------------------------------------------------------------------------------- > > Key: HIVE-25521 > URL: https://issues.apache.org/jira/browse/HIVE-25521 > Project: Hive > Issue Type: Bug > Reporter: Harish JP > Assignee: Harish JP > Priority: Major > Time Spent: 10m > Remaining Estimate: 0h > > Currently if files of different compressions are in same directory then > concatenate can fail and cause data corruption. This happens because file can > be moved by one task as incompatible file and the other tasks will fail after > this. > > This issue is addressed in this Jira by only processing a file in one task > where offset 0 is process and ignoring the the file in all other tasks. -- This message was sent by Atlassian Jira (v8.3.4#803005)