[jira] [Commented] (DRILL-6850) JDBC integration tests failures

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DRILL-6850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16688259#comment-16688259 ] 

ASF GitHub Bot commented on DRILL-6850:
---------------------------------------

vvysotskyi opened a new pull request #1542: DRILL-6850: JDBC integration tests failures
URL: https://github.com/apache/drill/pull/1542
 
 
   In the first commit:
   - Fixed RDBMS integration tests (expected decimal output and testCrossSourceMultiFragmentJoin)
   - Updated libraries versions
   - Resolved NPE for empty result
   
   In the second commit:
   - Made changes to force setting DRILL_LOGICAL Convention for DrillRelFactories and DrillFilterRel
   - Fixed workspace case insensitivity for JDBC storage plugin
   
   In the third commit:
   - Allowed configuring table names case sensitivity for JDBC storage plugin

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> JDBC integration tests failures
> -------------------------------
>
>                 Key: DRILL-6850
>                 URL: https://issues.apache.org/jira/browse/DRILL-6850
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - JDBC
>    Affects Versions: 1.14.0
>            Reporter: Vitalii Diravka
>            Priority: Major
>             Fix For: 1.15.0
>
>
> The following command will run Drill integratiom tests for RDBMS (Derby and MySQL):
> _mvn integration-test failsafe:integration-test -pl contrib/storage-jdbc_
> Currently some drill/exec/store/jdbc TestJdbcPluginWithDerbyIT and TestJdbcPluginWithMySQLIT tests fail:
> {code}
> Results :
> Failed tests: 
>   TestJdbcPluginWithDerbyIT.showTablesDefaultSchema:117 expected:<1> but was:<0>
> Tests in error: 
>   TestJdbcPluginWithDerbyIT.describe » UserRemote VALIDATION ERROR: Unknown tabl...
>   TestJdbcPluginWithDerbyIT.pushdownDoubleJoinAndFilter:111->PlanTestBase.testPlanMatchingPatterns:84->PlanTestBase.testPlanMatchingPatterns:89->PlanTestBase.getPlanInString:369->BaseTestQuery.testSqlWithResults:322->BaseTestQuery.testRunAndReturn:341 » Rpc
>   TestJdbcPluginWithDerbyIT.testCrossSourceMultiFragmentJoin » UserRemote VALIDA...
>   TestJdbcPluginWithDerbyIT.validateResult:71 »  at position 0 column '`NUMERIC_...
>   TestJdbcPluginWithMySQLIT.validateResult:108 »  at position 0 column '`numeric...
> Tests run: 14, Failures: 1, Errors: 5, Skipped: 0
> {code} 
> Most likely these are old regressions.
> Additionally NPE for empty result is resolved:
> http://drill.apache.org/blog/2018/08/05/drill-1.14-released/#comment-4082559169
> Also, in the scope of this Jira will be allowed configuring table names case sensitivity for JDBC storage plugin. Some databases, like derby or postgres are case insensitive regarding table names, but other databases like mySQL(depending on its configuration) may be case sensitive regarding table names. Therefore will be provided a parameter for storage plugin {{caseInsensitiveTableNames}} which determines whether table names should be handled considering case sensitivity.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)