Github user vvysotskyi commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1232#discussion_r183998600
  
    --- Diff: 
contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcRecordReader.java
 ---
    @@ -225,10 +247,10 @@ public int next() {
         int counter = 0;
         Boolean b = true;
         try {
    -      while (counter < 4095 && b == true) { // loop at 4095 since 
nullables use one more than record count and we
    +      while (counter < 4095 && b) { // loop at 4095 since nullables use 
one more than record count and we
                                                 // allocate on powers of two.
             b = resultSet.next();
    -        if(b == false) {
    +        if(!b) {
    --- End diff --
    
    Thanks, done.


---

Reply via email to