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 A4B95200CC5 for ; Tue, 11 Jul 2017 09:14:21 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id A33711653E3; Tue, 11 Jul 2017 07:14:21 +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 E8ED81653DD for ; Tue, 11 Jul 2017 09:14:20 +0200 (CEST) Received: (qmail 20889 invoked by uid 500); 11 Jul 2017 07:14:19 -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 20878 invoked by uid 99); 11 Jul 2017 07:14:19 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 11 Jul 2017 07:14:19 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id 04D2C195B4A; Tue, 11 Jul 2017 07:14:19 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 3.251 X-Spam-Level: *** X-Spam-Status: No, score=3.251 tagged_above=-999 required=6.31 tests=[HEADER_FROM_DIFFERENT_DOMAINS=0.001, HTML_MESSAGE=2, KAM_LAZY_DOMAIN_SECURITY=1, KAM_LOTSOFHASH=0.25, RP_MATCHES_RCVD=-0.001, URIBL_BLOCKED=0.001] autolearn=disabled Received: from mx1-lw-eu.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id 7deQln68PE5X; Tue, 11 Jul 2017 07:14:17 +0000 (UTC) Received: from mailrelay1-us-west.apache.org (mailrelay1-us-west.apache.org [209.188.14.139]) by mx1-lw-eu.apache.org (ASF Mail Server at mx1-lw-eu.apache.org) with ESMTP id 16242623EC; Tue, 11 Jul 2017 07:13:45 +0000 (UTC) Received: from reviews.apache.org (unknown [10.41.0.12]) by mailrelay1-us-west.apache.org (ASF Mail Server at mailrelay1-us-west.apache.org) with ESMTP id 619CBE0069; Tue, 11 Jul 2017 07:13:44 +0000 (UTC) Received: from reviews-vm2.apache.org (localhost [IPv6:::1]) by reviews.apache.org (ASF Mail Server at reviews-vm2.apache.org) with ESMTP id E7DA9C401AA; Tue, 11 Jul 2017 07:13:43 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============5312178613426400572==" MIME-Version: 1.0 Subject: Re: Review Request 60728: HIVE-16989: fix some alerts found by lgtm.com From: Malcolm Taylor To: Malcolm Taylor , hive Date: Tue, 11 Jul 2017 07:13:43 -0000 Message-ID: <20170711071343.38508.58031@reviews-vm2.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: Malcolm Taylor X-ReviewGroup: hive X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/60728/ X-Sender: Malcolm Taylor References: <20170710081333.6125.67407@reviews-vm2.apache.org> In-Reply-To: <20170710081333.6125.67407@reviews-vm2.apache.org> X-ReviewBoard-Diff-For: ql/src/test/org/apache/hadoop/hive/ql/exec/TestKeyWrapperFactory.java Reply-To: Malcolm Taylor X-ReviewRequest-Repository: hive-git archived-at: Tue, 11 Jul 2017 07:14:21 -0000 --===============5312178613426400572== 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/60728/ ----------------------------------------------------------- (Updated July 11, 2017, 7:13 a.m.) Review request for hive. Summary (updated) ----------------- HIVE-16989: fix some alerts found by lgtm.com Repository: hive-git Description ------- HIVE-16989 fix some alerts found by lgtm.com fix TextKeyWrapper.equals and add tests add .reviewboardrc Diffs ----- .reviewboardrc 49231678a72bc37efe052bf2d41006f3b595f7ab beeline/src/java/org/apache/hive/beeline/BeeLine.java 33a65d49656c77cf0736df0c53bd32c13fea6173 llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/StaticPermanentFunctionChecker.java 15968fa95e40a17d575c546c4880440b7bd93fd0 metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreSchemaInfo.java d6d83e03ac583a5f0e6e9144595e3999786d1648 metastore/src/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java 970038dd6126732840f1a8619fcf7efaa742b956 ql/src/java/org/apache/hadoop/hive/ql/exec/KeyWrapperFactory.java 5154a5fbe99c37bd2279f991a5e286a9c04918a4 ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/FlatRowContainer.java 9b1af1bd38e694c7f670aee0508cfb832fd109b7 ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/Repeated.java a7fad7177e39ef9112f08da0e7e6cc714ef96b8a ql/src/java/org/apache/hadoop/hive/ql/parse/repl/load/DumpMetaData.java 2d5e3b168b7cb2c3cf323533790af4bd87a333b8 ql/src/java/org/apache/hadoop/hive/ql/plan/TezEdgeProperty.java 5d7ddc880a566e63237e4b76fd1d6e614d502974 ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFMkCollectionEvaluator.java 2b5e6dd31d2b1c58d55a6e2bb1eb615748bc83e0 ql/src/test/org/apache/hadoop/hive/ql/exec/TestKeyWrapperFactory.java PRE-CREATION Diff: https://reviews.apache.org/r/60728/diff/1/ Testing ------- Ran tests. Some tests failed, but also appear to fail on master. Added tests for equals() methods in KeyWrapper classes, which fail on master but pass with this patch. Thanks, Malcolm Taylor --===============5312178613426400572==--