[ https://issues.apache.org/jira/browse/HIVE-24308?focusedWorklogId=504139&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-504139 ]
ASF GitHub Bot logged work on HIVE-24308: ----------------------------------------- Author: ASF GitHub Bot Created on: 23/Oct/20 12:49 Start Date: 23/Oct/20 12:49 Worklog Time Spent: 10m Work Description: pgaref commented on pull request #1604: URL: https://github.com/apache/hive/pull/1604#issuecomment-715320363 @rbalamohan @jesus Can you please take a look? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 504139) Time Spent: 20m (was: 10m) > FIX conditions used for DPHJ conversion > ----------------------------------------- > > Key: HIVE-24308 > URL: https://issues.apache.org/jira/browse/HIVE-24308 > Project: Hive > Issue Type: Bug > Reporter: Panagiotis Garefalakis > Assignee: Panagiotis Garefalakis > Priority: Major > Labels: pull-request-available > Time Spent: 20m > Remaining Estimate: 0h > > Found a weird scenario when looking at the ConvertJoinMapJoin logic: > [https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java#L1198] > When the distinct keys cannot fit in memory AND the DPHJ ShuffleSize is > lower than expected the code returns a MJ because of the condition above! > In general, I believe the ShuffleSize check: > [https://github.com/apache/hive/blob/master/ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java#L1624] > should be part of the shuffleJoin DPHJ conversion. > And the preferred conversion would be: MJ > DPHJ > SMB -- This message was sent by Atlassian Jira (v8.3.4#803005)