[GitHub] drill pull request #1213: Minor code cleanup

2018-04-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r182445689 --- Diff: exec/vector/src/test/java/org/apache/drill/exec/vector/VariableLengthVectorTest.java --- @@ -1,4 +1,4 @@ -/** +/* --- End diff --

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r182440952 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/vector/complex/writer/TestPromotableWriter.java --- @@ -1,4 +1,4 @@ -/** +/* ---

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r182445146 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/exception/OversizedAllocationException.java --- @@ -27,6 +27,7 @@ * {@link

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r182444596 --- Diff: exec/java-exec/src/test/java/org/apache/drill/exec/vector/complex/writer/TestPromotableWriter.java --- @@ -33,6 +33,7 @@ public void list()

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-18 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r182445595 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/util/CallBack.java --- @@ -1,4 +1,4 @@ -/** +/* --- End diff -- The same as

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-16 Thread vrozov
Github user vrozov commented on a diff in the pull request: https://github.com/apache/drill/pull/1213#discussion_r181932078 --- Diff: exec/vector/src/main/java/org/apache/drill/exec/exception/OversizedAllocationException.java --- @@ -1,4 +1,4 @@ -/** +/* --- End diff

[GitHub] drill pull request #1213: Minor code cleanup

2018-04-16 Thread paul-rogers
GitHub user paul-rogers opened a pull request: https://github.com/apache/drill/pull/1213 Minor code cleanup Pulled the remaining code cleanup items out of the Result Set work into this simple PR. You can merge this pull request into a Git repository by running: $ git pull