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

2018-03-15 Thread kkhatua
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 n

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

2018-03-16 Thread kfaraaz
Github user kfaraaz commented on the issue: https://github.com/apache/drill/pull/1159 I don't know about the other file, I didn't add it. Let me check. Thanks, Khurram ---