Return-Path: X-Original-To: apmail-hive-dev-archive@www.apache.org Delivered-To: apmail-hive-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id C2A3D17B91 for ; Tue, 20 Jan 2015 20:23:38 +0000 (UTC) Received: (qmail 82746 invoked by uid 500); 20 Jan 2015 20:23:38 -0000 Delivered-To: apmail-hive-dev-archive@hive.apache.org Received: (qmail 82674 invoked by uid 500); 20 Jan 2015 20:23:38 -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 82655 invoked by uid 99); 20 Jan 2015 20:23:37 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 20 Jan 2015 20:23:37 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 5DB821D4822; Tue, 20 Jan 2015 20:23:34 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============7702016749236124156==" MIME-Version: 1.0 Subject: Review Request 30086: HIVE-9327 From: =?utf-8?q?Jes=C3=BAs_Camacho_Rodr=C3=ADguez?= To: =?utf-8?q?Jes=C3=BAs_Camacho_Rodr=C3=ADguez?= , hive Date: Tue, 20 Jan 2015 20:23:34 -0000 Message-ID: <20150120202334.9402.35744@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: =?utf-8?q?Jes=C3=BAs_Camacho_Rodr=C3=ADguez?= X-ReviewGroup: hive X-ReviewRequest-URL: https://reviews.apache.org/r/30086/ X-Sender: =?utf-8?b?Ikplc8O6cyBDYW1hY2hvIFJvZHLDrWd1ZXoiIDxub3JlcGx5QHJldmlld3Mu?= =?utf-8?q?apache=2Eorg=3E?= Reply-To: =?utf-8?q?Jes=C3=BAs_Camacho_Rodr=C3=ADguez?= X-ReviewRequest-Repository: hive-git --===============7702016749236124156== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30086/ ----------------------------------------------------------- Review request for hive. Bugs: HIVE-9327 https://issues.apache.org/jira/browse/HIVE-9327 Repository: hive-git Description ------- ParseContext includes a map of Operator to RowResolver (OpParseContext). It would be ideal to remove this. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnInfo.java a34a31d5dde99896c809ea481db4ff6144426da7 ql/src/java/org/apache/hadoop/hive/ql/exec/RowSchema.java 450d7f364d1d83facf58e85a9bce87029b53d0d9 ql/src/java/org/apache/hadoop/hive/ql/exec/SelectOperator.java 1dbcb067c7c60d5b05141d0c59c6a59fa5b7418d ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 9ed2c61bcf2be97b7f89f52887bc36a7780898f5 ql/src/java/org/apache/hadoop/hive/ql/optimizer/AbstractSMBJoinProc.java a948b19a18125260df51ba8655226ed4efec510c ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPruner.java 046a52f85efc41d65ee6e140e124d5e1d21494fc ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPrunerProcCtx.java 5d848a1291d2292a0a569e9e88c32bc6a7a0d646 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPrunerProcFactory.java abf32f179911934e48906d2a59b9d87952e77a13 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagate.java 14e20dd6966b87d8d3fbfcaffbfe1f1a4d6b8127 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagateProcCtx.java 91af3aa5c70eb870fe692e9be0ec533b51ff0439 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConstantPropagateProcFactory.java d692e8e5267c513c70efd978b2f2a6e2c14007c5 ql/src/java/org/apache/hadoop/hive/ql/optimizer/ConvertJoinMapJoin.java 567c42e4c8f69c9276ebd202a7a2fc02f28d6ccc ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java b00fa52d03ebd66cc60793bb1e236ee7c18ac69a ql/src/java/org/apache/hadoop/hive/ql/optimizer/MapJoinProcessor.java d849dcf33f4d0d48696ff227a7594b330f5c8b63 ql/src/java/org/apache/hadoop/hive/ql/optimizer/NonBlockingOpDeDupProc.java 175a53cd084e3e628a17365c826c8ae09c1ab4dc ql/src/java/org/apache/hadoop/hive/ql/optimizer/ReduceSinkMapJoinProc.java ed6f7132077144327642fece0a0737b3ea71cbe7 ql/src/java/org/apache/hadoop/hive/ql/optimizer/SkewJoinOptimizer.java 28d8201654f6080987f83775c4326ea16fe5a141 ql/src/java/org/apache/hadoop/hive/ql/optimizer/SortedDynPartitionOptimizer.java e16ba6c004dc5b3ce734c32d977fd0985b5d1207 ql/src/java/org/apache/hadoop/hive/ql/optimizer/SparkMapJoinProcessor.java c69d492ea0fda9a9a2460e88725ea498dce48ddd ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/CorrelationOptimizer.java 20655c1406c70b060a19eea7b6b57b726b5490ea ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/CorrelationUtilities.java dc906e8fa3a19cc5aa169c1fe5a8f619807efb38 ql/src/java/org/apache/hadoop/hive/ql/optimizer/correlation/QueryPlanTreeTransformation.java 080725b933c6574c81a181cac960a4a739b576f6 ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteGBUsingIndex.java 06a9478283437dd020cd199e139bc3eedb9c15f1 ql/src/java/org/apache/hadoop/hive/ql/optimizer/index/RewriteQueryUsingAggregateIndexCtx.java 72f458830a8474e6d4aa9d5788c8de509185d34f ql/src/java/org/apache/hadoop/hive/ql/optimizer/lineage/ExprProcCtx.java d3caaf0b8f1859cbd246f2b9782d49bb2deb42ff ql/src/java/org/apache/hadoop/hive/ql/optimizer/lineage/ExprProcFactory.java fdbb93ed07648cc6720b0d00981a8255a077191f ql/src/java/org/apache/hadoop/hive/ql/optimizer/lineage/OpProcFactory.java d6a6ed6709b8a280e490909ffd499f7ea48dc77a ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/CommonJoinTaskDispatcher.java 19503dc0262988fb5bed86c8a12326ebc529c7c4 ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SortMergeJoinTaskDispatcher.java a135cf50fdce1e407192f9f634172f7ff12bd0aa ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkMapJoinOptimizer.java 9ff47c73a52d4b6319eb2dfae64b01bdcdf671fd ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezWork.java 6a87929b70df81823ed0d4fbef92dc4e6f1acdb9 ql/src/java/org/apache/hadoop/hive/ql/parse/ParseContext.java b838bff598bdc6c8d4c2728967d2c2bf0ee63e9e ql/src/java/org/apache/hadoop/hive/ql/parse/RowResolver.java 469dc9f4062387580c98bebc61e2c972b5d32d84 ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 4364f2830d35b89266bf79948263dd64998fe5cc ql/src/java/org/apache/hadoop/hive/ql/parse/TaskCompiler.java f2eb4d27fca6472a9d4b777a54fcce8a729b3cd6 ql/src/java/org/apache/hadoop/hive/ql/plan/MapWork.java 9abb8db04fc2d6fced7f4e0aa4628df8575e2237 ql/src/java/org/apache/hadoop/hive/ql/ppd/ExprWalkerInfo.java 9bed527d15cf50c60fecded8074228657bfb7e58 ql/src/java/org/apache/hadoop/hive/ql/ppd/ExprWalkerProcFactory.java 8a8b0d5be0fe884693f370557e3b3e8a822b8f97 ql/src/java/org/apache/hadoop/hive/ql/ppd/OpProcFactory.java 0a7a757a6391079c6e068defea497576bb2a09bf ql/src/java/org/apache/hadoop/hive/ql/ppd/OpWalkerInfo.java 2b4e4acb9d24ac63b80eebda815a977862ac3aa6 ql/src/java/org/apache/hadoop/hive/ql/ppd/PredicateTransitivePropagate.java 00952b7b62070711778bad5594a7440bbc1b766d ql/src/java/org/apache/hadoop/hive/ql/ppd/SyntheticJoinPredicate.java fe59c132f1e343028205386700a746d72f3fed05 Diff: https://reviews.apache.org/r/30086/diff/ Testing ------- Existing tests. Thanks, Jesús Camacho Rodríguez --===============7702016749236124156==--