[ https://issues.apache.org/jira/browse/HIVE-21932?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16874671#comment-16874671 ]
Hive QA commented on HIVE-21932: -------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12973122/HIVE-21932.01.patch {color:red}ERROR:{color} -1 due to no test(s) being added or modified. {color:green}SUCCESS:{color} +1 due to 16357 tests passed Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/17783/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/17783/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-17783/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.YetusPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase {noformat} This message is automatically generated. ATTACHMENT ID: 12973122 - PreCommit-HIVE-Build > IndexOutOfRangeExeption in FileChksumIterator > --------------------------------------------- > > Key: HIVE-21932 > URL: https://issues.apache.org/jira/browse/HIVE-21932 > Project: Hive > Issue Type: Bug > Reporter: Vihang Karajgaonkar > Assignee: Vihang Karajgaonkar > Priority: Major > Attachments: HIVE-21932.01.patch > > > According to definition of {{InsertEventRequestData}} in > {{hive_metastore.thrift}} the {{filesAddedChecksum}} is a optional field. But > the FileChksumIterator does not handle it correctly when a client fires a > insert event which does not have file checksums. The issue is that > {{InsertEvent}} class initializes fileChecksums list to a empty arrayList to > the following check will never come into play > {noformat} > result = ReplChangeManager.encodeFileUri(files.get(i), chksums != null ? > chksums.get(i) : null, > subDirs != null ? subDirs.get(i) : null); > {noformat} > The chksums check above should include a {{!chksums.isEmpty()}} check as well > in the above line. -- This message was sent by Atlassian JIRA (v7.6.3#76005)