gracewalkuski opened a new issue #6557:
URL: https://github.com/apache/incubator-pinot/issues/6557


   Add the implementation for the `execute()` function in the 
[PinotPreparedStatement](https://github.com/apache/incubator-pinot/blob/release-0.6.0/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotPreparedStatement.java)
 class. 
   
   ```
   @Override
   public ResultSet execute()
       throws SQLException {
     executeQuery();
   }
   ```
   
   This will allow for apps using the JDBC to use the `execute()` function 
without any special configuration to call executeQuery() instead. Currently, it 
calls the execute function 
[here](https://github.com/apache/incubator-pinot/blob/e892cb2942f1f41ac7056d977eeca0e5b22897d0/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/base/AbstractBasePreparedStatement.java#L97)
 which caused a lot of confusion initially. 
   
   Specifically, this allows [Jooq](https://www.jooq.org/) to call Pinot 
correctly with its `.fetch()` method.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to