You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by kkhatua <gi...@git.apache.org> on 2018/03/15 19:15:05 UTC

[GitHub] drill issue #1159: DRILL-6215: Changed Statement to PreparedStatement in Jdb...

Github user kkhatua commented on the issue:

    https://github.com/apache/drill/pull/1159
  
    @kfaraaz Are there unit tests specific to the JDBC Storage Plugin? They're not the same as the JDBC unit tests, which test the Drill JDBC driver.
    
    While the change seems straightforward, I'm not sure if all JDBC drivers support PreparedStatement. For e.g., within Drill, the PreparedStatement is executed as a standard Statement object, which is why this _works_ functionally.
    
    What happens if a JDBC driver backing the JDBC storage plugin does not support PreparedStatement (i.e. it is a No-Op)? 


---