[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/drill/pull/1161 ---

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-31 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r178445472 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/rowSet/test/TestHyperVectorReaders.java --- @@ -0,0 +1,365 @@ +/* + * Licensed to t

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-31 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r178445424 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/rowSet/HyperRowSetImpl.java --- @@ -45,8 +50,67 @@ public RowSetReader buildReader(HyperRow

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-31 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r177833383 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/model/ReaderIndex.java --- @@ -28,26 +28,30 @@ public abstract c

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-31 Thread paul-rogers
Github user paul-rogers commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r177846274 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/rowSet/HyperRowSetImpl.java --- @@ -45,8 +50,67 @@ public RowSetReader buildReader(HyperRow

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-28 Thread ppadma
Github user ppadma commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r177564887 --- Diff: exec/java-exec/src/main/java/org/apache/drill/exec/physical/rowSet/model/ReaderIndex.java --- @@ -28,26 +28,30 @@ public abstract class

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-28 Thread ppadma
Github user ppadma commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r177182331 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/rowSet/test/TestHyperVectorReaders.java --- @@ -0,0 +1,365 @@ +/* + * Licensed to the Ap

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-28 Thread ppadma
Github user ppadma commented on a diff in the pull request: https://github.com/apache/drill/pull/1161#discussion_r177175974 --- Diff: exec/java-exec/src/test/java/org/apache/drill/test/rowSet/HyperRowSetImpl.java --- @@ -45,8 +50,67 @@ public RowSetReader buildReader(HyperRowSet r

[GitHub] drill pull request #1161: DRILL-6230: Extend row set readers to handle hyper...

2018-03-11 Thread paul-rogers
GitHub user paul-rogers opened a pull request: https://github.com/apache/drill/pull/1161 DRILL-6230: Extend row set readers to handle hyper vectors The current row set readers have incomplete support for hyper-vectors. To add full support, we need an interface that supports either s