[jira] [Created] (DRILL-5954) ListVector derives from BaseRepeatedValueVector, shadows offsets

2017-11-11 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5954: -- Summary: ListVector derives from BaseRepeatedValueVector, shadows offsets Key: DRILL-5954 URL: https://issues.apache.org/jira/browse/DRILL-5954 Project: Apache Drill

[GitHub] drill issue #518: DRILL-4653.json - Malformed JSON should not stop the entir...

2017-11-11 Thread paul-rogers
Github user paul-rogers commented on the issue: https://github.com/apache/drill/pull/518 Further testing revealed limitations in the underlying Jackson parser. That parser will not recover from other errors such as: ``` { a: } ``` See

[jira] [Created] (DRILL-5953) JSON reader unable to recover from most parse errors

2017-11-11 Thread Paul Rogers (JIRA)
Paul Rogers created DRILL-5953: -- Summary: JSON reader unable to recover from most parse errors Key: DRILL-5953 URL: https://issues.apache.org/jira/browse/DRILL-5953 Project: Apache Drill Issue

Re: gitbox?

2017-11-11 Thread Abhishek Girish
+1 On Wed, Nov 1, 2017 at 3:51 AM, Arina Yelchiyeva wrote: > +1 but as we have communicated on Hangouts committer should be responsible > for: > > 1. making sure that there is only one commit (otherwise ask developer to > squash the commits, unless several commits

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379534 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379448 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/DrillKafkaConfig.java --- @@ -0,0 +1,31 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379457 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379539 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379549 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaRecordReader.java --- @@ -0,0 +1,178 @@ +/** + * Licensed

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379428 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379256 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaRecordReader.java --- @@ -0,0 +1,178 @@ +/** + * Licensed

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379243 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/DrillKafkaConfig.java --- @@ -0,0 +1,31 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379142 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379209 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + + +http://maven.apache.org/POM/4.0.0

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150379252 --- Diff: contrib/storage-kafka/src/main/java/org/apache/drill/exec/store/kafka/KafkaGroupScan.java --- @@ -0,0 +1,301 @@ +/** + * Licensed to

[GitHub] drill pull request #1027: DRILL-4779 : Kafka storage plugin

2017-11-11 Thread akumarb2010
Github user akumarb2010 commented on a diff in the pull request: https://github.com/apache/drill/pull/1027#discussion_r150376627 --- Diff: contrib/storage-kafka/pom.xml --- @@ -0,0 +1,130 @@ + + +http://maven.apache.org/POM/4.0.0