[GitHub] drill pull request #656: DRILL-5034: Select timestamp from hive generated pa...

2017-01-26 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/656#discussion_r98070065 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetReaderUtility.java --- @@ -323,18 +323,28 @@ public static

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-30 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r98512248 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-01-25 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r97848365 --- Diff: contrib/native/client/src/clientlib/saslAuthenticatorImpl.cpp --- @@ -0,0 +1,207 @@ +/* + * Licensed to the Apache Software Foundation

[GitHub] drill pull request #578: DRILL-4280: Kerberos Authentication

2017-02-13 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/578#discussion_r100938180 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.hpp --- @@ -362,6 +363,7 @@ class DrillClientImpl : public DrillClientImplBase

[GitHub] drill issue #600: DRILL-4373: Drill and Hive have incompatible timestamp rep...

2016-10-06 Thread bitblender
Github user bitblender commented on the issue: https://github.com/apache/drill/pull/600 I can't see NullableFixedByteAlignedReaders.java. Shows up as a binary file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] drill pull request #600: DRILL-4373: Drill and Hive have incompatible timest...

2016-10-06 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/600#discussion_r82314071 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetReaderUtility.java --- @@ -45,4 +53,34 @@ public static int

[GitHub] drill issue #634: DRILL-4974: NPE in FindPartitionConditions.analyzeCall() f...

2016-10-27 Thread bitblender
Github user bitblender commented on the issue: https://github.com/apache/drill/pull/634 @amansinha100 Can you please review this change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] drill pull request #600: DRILL-4373: Drill and Hive have incompatible timest...

2016-10-17 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/600#discussion_r83761501 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/ParquetReaderUtility.java --- @@ -45,4 +53,34 @@ public static int

[GitHub] drill pull request #634: DRILL-4974: NPE in FindPartitionConditions.analyzeC...

2016-10-27 Thread bitblender
GitHub user bitblender opened a pull request: https://github.com/apache/drill/pull/634 DRILL-4974: NPE in FindPartitionConditions.analyzeCall() for 'holistic' expressions Changes: Added a missing null check in FindPartitionConditions.analyzeCall(), to ensure that opStack.peek

[GitHub] drill pull request #690: DRILL-5121 A memory leak is observed when exact cas...

2016-12-10 Thread bitblender
GitHub user bitblender opened a pull request: https://github.com/apache/drill/pull/690 DRILL-5121 A memory leak is observed when exact case is not specified for a column in a filter condition Fix for https://issues.apache.org/jira/browse/DRILL-5121. Changes fieldVectorMap

[GitHub] drill pull request #638: DRILL-4982: Separate Hive reader classes for differ...

2016-12-01 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/638#discussion_r90574793 --- Diff: contrib/storage-hive/core/src/main/java/org/apache/drill/exec/store/hive/HiveAbstractReader.java --- @@ -218,17 +229,18 @@ private void init

[GitHub] drill pull request #697: DRILL-5097: Using store.parquet.reader.int96_as_tim...

2017-01-02 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/697#discussion_r94350780 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/NullableFixedByteAlignedReaders.java --- @@ -110,9 +110,14

[GitHub] drill pull request #697: DRILL-5097: Using store.parquet.reader.int96_as_tim...

2017-01-02 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/697#discussion_r94350967 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/store/parquet/columnreaders/NullableFixedByteAlignedReaders.java --- @@ -132,6 +137,9

[GitHub] drill pull request #695: DRILL-4868: fix how hive function set DrillBuf.

2016-12-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/695#discussion_r93133495 --- Diff: contrib/storage-hive/core/src/main/codegen/templates/ObjectInspectorHelper.java --- @@ -172,48 +172,35 @@ public static JBlock getDrillObject

[GitHub] drill pull request #783: DRILL-5324: Provide simplified column reader/writer...

2017-04-13 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/783#discussion_r111289506 --- Diff: exec/vector/src/main/codegen/templates/ColumnAccessors.java --- @@ -0,0 +1,333 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[GitHub] drill pull request #783: DRILL-5324: Provide simplified column reader/writer...

2017-04-13 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/783#discussion_r111495333 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/vector/accessor/ScalarWriter.java --- @@ -0,0 +1,32 @@ +/* + * Licensed to the Apache

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-13 Thread bitblender
GitHub user bitblender opened a pull request: https://github.com/apache/drill/pull/875 DRILL-5671 Set secure ACLs (Access Control List) for Drill ZK nodes in a secure cluster You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-13 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/875#discussion_r127367344 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKSecureACLProvider.java --- @@ -0,0 +1,71 @@ +/** + * Licensed

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-13 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/875#discussion_r127367354 --- Diff: exec/java-exec/src/main/resources/drill-module.conf --- @@ -112,7 +112,8 @@ drill.exec: { retry: { count: 7200

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-14 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/875#discussion_r127540793 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/coord/zk/ZKACLProviderFactory.java --- @@ -0,0 +1,44 @@ +/** + * Licensed

[GitHub] drill pull request #875: DRILL-5671 Set secure ACLs (Access Control List) fo...

2017-07-14 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/875#discussion_r127540711 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/ExecConstants.java --- @@ -39,6 +39,7 @@ String ZK_TIMEOUT = "drill.exec.zk.ti

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131459203 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/ResultSetLoaderImpl.java --- @@ -0,0 +1,412 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131684173 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/TupleSetImpl.java --- @@ -0,0 +1,551 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131685349 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/TupleSetImpl.java --- @@ -0,0 +1,551 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131554894 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/ResultSetLoaderImpl.java --- @@ -0,0 +1,412 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131216670 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/ColumnLoaderImpl.java --- @@ -0,0 +1,31 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r130429208 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/TupleSchema.java --- @@ -0,0 +1,91 @@ +/* + * Licensed to the Apache

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131564509 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/ResultVectorCache.java --- @@ -0,0 +1,181 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r131284158 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/LogicalTupleLoader.java --- @@ -0,0 +1,204 @@ +/* + * Licensed

[GitHub] drill pull request #866: DRILL-5657: Implement size-aware result set loader

2017-08-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/866#discussion_r130250994 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/ResultSetLoader.java --- @@ -0,0 +1,170 @@ +/* + * Licensed

[GitHub] drill issue #875: DRILL-5671 Set secure ACLs (Access Control List) for Drill...

2017-08-07 Thread bitblender
Github user bitblender commented on the issue: https://github.com/apache/drill/pull/875 Yes. There will be a commit addressing the issues raised here. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] drill pull request #894: DRILL-5701: drill.connections.rpc.

2017-08-22 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/894#discussion_r134553446 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestUserBitKerberos.java --- @@ -137,6 +142,41 @@ public Void run() throws

[GitHub] drill pull request #840: DRILL-5517: Size-aware set methods in value vectors

2017-06-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/840#discussion_r122526465 --- Diff: exec/vector/src/main/codegen/templates/FixedValueVectors.java --- @@ -806,10 +998,32 @@ public void generateTestDataAlt(int size

[GitHub] drill pull request #840: DRILL-5517: Size-aware set methods in value vectors

2017-06-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/840#discussion_r118806486 --- Diff: exec/memory/base/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java --- @@ -174,6 +175,40 @@ public ByteBuf setDouble(int index, double

[GitHub] drill pull request #840: DRILL-5517: Size-aware set methods in value vectors

2017-06-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/840#discussion_r122533121 --- Diff: exec/vector/src/main/codegen/templates/VariableLengthVectors.java --- @@ -548,6 +567,23 @@ public void setSafe(int index, ByteBuffer bytes, int

[GitHub] drill pull request #840: DRILL-5517: Size-aware set methods in value vectors

2017-06-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/840#discussion_r122541355 --- Diff: exec/memory/base/src/main/java/io/netty/buffer/UnsafeDirectLittleEndian.java --- @@ -174,6 +175,40 @@ public ByteBuf setDouble(int index, double

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r117412175 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -234,6 +233,37 @@ void DrillClientImpl::Close

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-19 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r117412544 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -364,7 +395,41 @@ connectionStatus_t DrillClientImpl::recvHandshake

[GitHub] drill pull request #837: DRILL-5514: Enhance VectorContainer to merge two ro...

2017-06-15 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/837#discussion_r122287096 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/record/TestVectorContainer.java --- @@ -110,13 +110,16 @@ public void testContainerMerge

[GitHub] drill pull request #837: DRILL-5514: Enhance VectorContainer to merge two ro...

2017-06-15 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/837#discussion_r122287615 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/record/BatchSchema.java --- @@ -162,20 +162,22 @@ private boolean majorTypeEqual(MajorType t1

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114442641 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -245,6 +264,64 @@ connectionStatus_t DrillClientImpl::sendSync(rpc

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115369295 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115371963 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114442242 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -245,6 +264,64 @@ connectionStatus_t DrillClientImpl::sendSync(rpc

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114442866 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -245,6 +264,64 @@ connectionStatus_t DrillClientImpl::sendSync(rpc

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114442367 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -245,6 +264,64 @@ connectionStatus_t DrillClientImpl::sendSync(rpc

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115370795 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114623038 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114629588 --- Diff: contrib/native/client/src/clientlib/utils.cpp --- @@ -111,4 +111,52 @@ AllocatedBuffer::~AllocatedBuffer(){ m_bufSize = 0

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115371638 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114778278 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -495,26 +612,45 @@ connectionStatus_t DrillClientImpl::handleAuthentication(const

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114828848 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115372156 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r11272 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -245,6 +264,64 @@ connectionStatus_t DrillClientImpl::sendSync(rpc

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114775522 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -370,6 +453,33 @@ void DrillClientImpl::handleHShakeReadTimeout(const boost

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r115369469 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114828323 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #809: Drill-4335: C++ client changes for supporting encry...

2017-05-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/809#discussion_r114818140 --- Diff: contrib/native/client/src/clientlib/drillClientImpl.cpp --- @@ -854,75 +990,328 @@ void DrillClientImpl::waitForResults

[GitHub] drill pull request #837: DRILL-5514: Enhance VectorContainer to merge two ro...

2017-06-05 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/837#discussion_r120198724 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/record/BatchSchema.java --- @@ -157,4 +158,26 @@ private boolean majorTypeEqual(MajorType t1

[GitHub] drill pull request #837: DRILL-5514: Enhance VectorContainer to merge two ro...

2017-06-05 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/837#discussion_r118797793 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/record/BatchSchema.java --- @@ -157,4 +158,26 @@ private boolean majorTypeEqual(MajorType t1

[GitHub] drill pull request #914: DRILL-5657: Size-aware vector writer structure

2017-09-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/914#discussion_r137851895 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/package-info.java --- @@ -0,0 +1,295 @@ +/* + * Licensed

[GitHub] drill pull request #914: DRILL-5657: Size-aware vector writer structure

2017-09-08 Thread bitblender
Github user bitblender commented on a diff in the pull request: https://github.com/apache/drill/pull/914#discussion_r137852118 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/impl/package-info.java --- @@ -0,0 +1,295 @@ +/* + * Licensed