[ https://issues.apache.org/jira/browse/HIVE-18873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16594410#comment-16594410 ]
Ankit Singhal commented on HIVE-18873: -------------------------------------- bq. Accumulo test needs to be looked at. [~vgarg] , I ran Accumulo qtests locally and they are passing for me with the fix as well. {code} [INFO] T E S T S [INFO] ------------------------------------------------------- [INFO] Running org.apache.hadoop.hive.cli.TestAccumuloCliDriver [INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 148.492 s - in org.apache.hadoop.hive.cli.TestAccumuloCliDriver [INFO] Running org.apache.hive.TestDummy [INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.042 s - in org.apache.hive.TestDummy [INFO] [INFO] Results: [INFO] [INFO] Tests run: 8, Failures: 0, Errors: 0, Skipped: 0 [INFO] {code} bq. Test results: [https://builds.apache.org/job/PreCommit-HIVE-Build/13437/testReport] Sorry for being late, testReport seems to be purged by now. Let me re-attach the patch for debugging. > Skipping predicate pushdown for MR silently at HiveInputFormat can cause > storage handlers to produce erroneous result > --------------------------------------------------------------------------------------------------------------------- > > Key: HIVE-18873 > URL: https://issues.apache.org/jira/browse/HIVE-18873 > Project: Hive > Issue Type: Bug > Affects Versions: 3.0.0 > Reporter: Ankit Singhal > Assignee: Ankit Singhal > Priority: Major > Attachments: HIVE-18873.2.patch, HIVE-18873.patch > > > {code:java} > // disable filter pushdown for mapreduce when there are more than one table > aliases, > // since we don't clone jobConf per alias > if (mrwork != null && mrwork.getAliases() != null && > mrwork.getAliases().size() > 1 && > jobConf.get(ConfVars.HIVE_EXECUTION_ENGINE.varname).equals("mr")) { > return; > } > {code} > I believe this needs to be handled at OpProcFactory so that hive doesn't > believe that predicate is handled by storage handler. -- This message was sent by Atlassian JIRA (v7.6.3#76005)