Return-Path: X-Original-To: apmail-drill-dev-archive@www.apache.org Delivered-To: apmail-drill-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 B0C01174F2 for ; Sun, 5 Apr 2015 23:49:50 +0000 (UTC) Received: (qmail 73641 invoked by uid 500); 5 Apr 2015 23:49:50 -0000 Delivered-To: apmail-drill-dev-archive@drill.apache.org Received: (qmail 73576 invoked by uid 500); 5 Apr 2015 23:49:50 -0000 Mailing-List: contact dev-help@drill.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@drill.apache.org Delivered-To: mailing list dev@drill.apache.org Received: (qmail 73561 invoked by uid 500); 5 Apr 2015 23:49:50 -0000 Delivered-To: apmail-incubator-drill-dev@incubator.apache.org Received: (qmail 73555 invoked by uid 99); 5 Apr 2015 23:49:50 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 05 Apr 2015 23:49:50 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 374B71C005D; Sun, 5 Apr 2015 23:49:48 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============1387701853279953602==" MIME-Version: 1.0 Subject: Re: Review Request 32867: DRILL-2695: Add support for large in lists via values operator From: "Jacques Nadeau" To: "Aman Sinha" , "Hanifi Gunes" Cc: "Jacques Nadeau" , "drill" Date: Sun, 05 Apr 2015 23:49:48 -0000 Message-ID: <20150405234948.5638.80194@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Jacques Nadeau" X-ReviewGroup: drill-git X-ReviewRequest-URL: https://reviews.apache.org/r/32867/ X-Sender: "Jacques Nadeau" References: <20150405211557.5637.87786@reviews.apache.org> In-Reply-To: <20150405211557.5637.87786@reviews.apache.org> Reply-To: "Jacques Nadeau" X-ReviewRequest-Repository: drill-git --===============1387701853279953602== 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/32867/ ----------------------------------------------------------- (Updated April 5, 2015, 11:49 p.m.) Review request for drill, Aman Sinha and Hanifi Gunes. Changes ------- Fix issues with timezone handling Repository: drill-git Description ------- Add Support for large in conditions through the use of the Values operator. Update JSON reader to support reading Extended JSON. Update JSON writer to support writing extended JSON data. Update JSON reader to automatically unwrap a file that includes a single top-level array. Update Options manager to use getOption(Validator) to directly retrieve typed value. Remove JSON rewinding Diffs (updated) ----- common/src/main/java/org/apache/drill/common/JSONOptions.java 945cd92 common/src/main/java/org/apache/drill/common/logical/data/Constant.java 460803d common/src/main/java/org/apache/drill/common/logical/data/Values.java PRE-CREATION common/src/main/java/org/apache/drill/common/logical/data/visitors/AbstractLogicalVisitor.java 92e370f common/src/main/java/org/apache/drill/common/logical/data/visitors/LogicalVisitor.java 3a426bf contrib/storage-mongo/src/main/java/org/apache/drill/exec/store/mongo/MongoRecordReader.java 15ef197 exec/java-exec/src/main/codegen/templates/JsonOutputRecordWriter.java 1d0dc9d exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java bd93206 exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/DateUtility.java a031bee exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/conv/JsonConvertFrom.java c828cf4 exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractPhysicalVisitor.java 27b0ecb exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/PhysicalVisitor.java e6a89d0 exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/Values.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/values/ValuesBatchCreator.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillOptiq.java 796f0f7 exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRuleSets.java b1a7189 exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillValuesRel.java a3551e7 exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/ScanFieldDeterminer.java 59c65f9 exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/ValuesPrel.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/ValuesPrule.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/server/options/BaseOptionManager.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/server/options/FallbackOptionManager.java 45d393c exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionManager.java 4ffe9a3 exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java 608fac7 exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JSONFormatPlugin.java 6cf1ce5 exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JSONRecordReader.java cc7cb83 exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JsonProcessor.java ce6017b exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JsonRecordWriter.java 76c4ace exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/RewindableUtf8Reader.java b9075de exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/reader/BaseJsonProcessor.java 509798a exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/reader/CountingJsonReader.java 1ef71e7 exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/BasicJsonOutput.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/DateOutputFormat.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/ExtendedJsonOutput.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/ExtendedType.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/ExtendedTypeName.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/JsonOutput.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/JsonReader.java 9738ff8 exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/JsonWriter.java de52b73 exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/VectorOutput.java PRE-CREATION exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/fn/WorkingBuffer.java PRE-CREATION exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/filter/TestLargeInClause.java PRE-CREATION exec/java-exec/src/test/java/org/apache/drill/exec/store/json/TestJsonRecordReader.java c4bfcce exec/java-exec/src/test/java/org/apache/drill/exec/vector/complex/writer/TestExtendedTypes.java PRE-CREATION exec/java-exec/src/test/resources/store/json/listdoc.json PRE-CREATION exec/java-exec/src/test/resources/vector/complex/extended.json PRE-CREATION Diff: https://reviews.apache.org/r/32867/diff/ Testing ------- Unit, Regression, SF100 Thanks, Jacques Nadeau --===============1387701853279953602==--