Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 64385200C00 for ; Wed, 4 Jan 2017 05:40:07 +0100 (CET) Received: by cust-asf.ponee.io (Postfix) id 630B8160B43; Wed, 4 Jan 2017 04:40:07 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 8C587160B46 for ; Wed, 4 Jan 2017 05:40:06 +0100 (CET) Received: (qmail 12862 invoked by uid 500); 4 Jan 2017 04:40:05 -0000 Mailing-List: contact dev-help@hive.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@hive.apache.org Delivered-To: mailing list dev@hive.apache.org Received: (qmail 12840 invoked by uid 99); 4 Jan 2017 04:40:05 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 04 Jan 2017 04:40:05 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 568B4310DCB; Wed, 4 Jan 2017 04:40:04 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============0836938707641561304==" MIME-Version: 1.0 Subject: Re: Review Request 55156: Min-max runtime filtering From: Deepak Jaiswal To: Jason Dere , Rajesh Balamohan , Gopal V , j.prasanth.j@gmail.com, Gunther Hagleitner Cc: Deepak Jaiswal , hive Date: Wed, 04 Jan 2017 04:40:04 -0000 Message-ID: <20170104044004.13569.90450@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Deepak Jaiswal X-ReviewGroup: hive X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/55156/ X-Sender: Deepak Jaiswal References: <20170103230310.13478.26802@reviews.apache.org> In-Reply-To: <20170103230310.13478.26802@reviews.apache.org> X-ReviewBoard-Diff-For: storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/LiteralDelegate.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/parse/RuntimeValuesInfo.java X-ReviewBoard-Diff-For: ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction.q.out X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/DynamicValueVectorExpression.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/exec/DynamicValueRegistry.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/plan/DynamicValue.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeDynamicValueDesc.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DynamicValueRegistryTez.java X-ReviewBoard-Diff-For: ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeDynamicValueEvaluator.java X-ReviewBoard-Diff-For: ql/src/test/queries/clientpositive/dynamic_semijoin_reduction.q Reply-To: Deepak Jaiswal X-ReviewRequest-Repository: hive-git archived-at: Wed, 04 Jan 2017 04:40:07 -0000 --===============0836938707641561304== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55156/ ----------------------------------------------------------- (Updated Jan. 4, 2017, 4:39 a.m.) Review request for hive, Gopal V, Gunther Hagleitner, Jason Dere, Prasanth_J, and Rajesh Balamohan. Changes ------- Fixed couple of build issues. The patch now applies cleanly. Bugs: HIVE-15269 https://issues.apache.org/jira/browse/HIVE-15269 Repository: hive-git Description ------- HIVE-15269 min-max runtime filtering. The patch also contains the patch for HIVE-15270. Diffs (updated) ----- common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 47db0c0 orc/src/java/org/apache/orc/impl/RecordReaderImpl.java 975804b orc/src/test/org/apache/orc/impl/TestRecordReaderImpl.java cdd62ac ql/src/java/org/apache/hadoop/hive/ql/exec/AbstractMapJoinOperator.java 69ba4a2 ql/src/java/org/apache/hadoop/hive/ql/exec/CommonJoinOperator.java 940f2dd ql/src/java/org/apache/hadoop/hive/ql/exec/DynamicValueRegistry.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeColumnEvaluator.java 24c8281 ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeConstantDefaultEvaluator.java 89a75eb ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeConstantEvaluator.java 4fe72a0 ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeDynamicValueEvaluator.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeEvaluator.java b8d6ab7 ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeEvaluatorFactory.java 0d03d8f ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeEvaluatorHead.java 42685fb ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeEvaluatorRef.java 0a6b66a ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeFieldEvaluator.java ff32626 ql/src/java/org/apache/hadoop/hive/ql/exec/ExprNodeGenericFuncEvaluator.java 221abd9 ql/src/java/org/apache/hadoop/hive/ql/exec/FilterOperator.java bd0d28c ql/src/java/org/apache/hadoop/hive/ql/exec/GroupByOperator.java 46f0ecd ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableSinkOperator.java ac5331e ql/src/java/org/apache/hadoop/hive/ql/exec/JoinUtil.java 9718c48 ql/src/java/org/apache/hadoop/hive/ql/exec/ObjectCache.java 440e0a1 ql/src/java/org/apache/hadoop/hive/ql/exec/ObjectCacheWrapper.java 9768efa ql/src/java/org/apache/hadoop/hive/ql/exec/SelectOperator.java 9049ddd ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ObjectCache.java 008f8a4 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DynamicValueRegistryTez.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/exec/tez/LlapObjectCache.java 0141230 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/MapRecordProcessor.java 955fa80 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/ObjectCache.java 06dca00 ql/src/java/org/apache/hadoop/hive/ql/exec/tez/ReduceRecordProcessor.java d80f201 ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorMapJoinOperator.java 0cb6c8a ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorSMBMapJoinOperator.java 80b0a14 ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java f6b6447 ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/DynamicValueVectorExpression.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/io/sarg/ConvertAstToSearchArg.java 9d900e4 ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java 26fcc45 ql/src/java/org/apache/hadoop/hive/ql/optimizer/FixedBucketPruningOptimizer.java 9e9beb0 ql/src/java/org/apache/hadoop/hive/ql/optimizer/RedundantDynamicPruningConditionsRemoval.java d9ce017 ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java aa1e509 ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java e2363eb ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java 35f34da ql/src/java/org/apache/hadoop/hive/ql/parse/RuntimeValuesInfo.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompiler.java e8b003e ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java cdb9e1b ql/src/java/org/apache/hadoop/hive/ql/plan/BaseWork.java 13a0811 ql/src/java/org/apache/hadoop/hive/ql/plan/DynamicValue.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeDynamicValueDesc.java PRE-CREATION ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestConvertAstToSearchArg.java 93b50a6 ql/src/test/org/apache/hadoop/hive/ql/io/sarg/TestSearchArgumentImpl.java 8cbc26d ql/src/test/org/apache/hadoop/hive/ql/optimizer/physical/TestVectorizer.java 3295372 ql/src/test/queries/clientpositive/dynamic_semijoin_reduction.q PRE-CREATION ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction.q.out PRE-CREATION storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/LiteralDelegate.java PRE-CREATION storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgumentFactory.java 8fda95c storage-api/src/java/org/apache/hadoop/hive/ql/io/sarg/SearchArgumentImpl.java 10d8c51 Diff: https://reviews.apache.org/r/55156/diff/ Testing ------- Thanks, Deepak Jaiswal --===============0836938707641561304==--