search

ActianCorp / actian_tableau_connector

Tableau connector (aka taco) for Actian Avalanche, Vector, and Ingres
https://extensiongallery.tableau.com/products/936
Apache License 2.0
1 stars 4 forks source link

Fix for TDVT test cases in tickets II-12247, II-12248, II-12251, II-12252, II-12253, II-12254 #39

Closed hab6 closed 11 months ago

hab6 commented 11 months ago

Summary

Remedy failures that occur in TDVT test cases using the Actian JDBC Connector.

TDVT Test Cases Fixed

Jira Ticket Test Set Test Name Without Fix With Fix
II-12247 logical.staples Query.Sort_Alphabetic_DESC_Top1 Failed Passed
II-12248 logical.union calcs.union Failed Passed
II-12251 logical.lod lod.calc.1.TopN Failed Passed
II-12252 logical.staples BUGS.B340 Failed Passed
II-12253 logical.calcs BUGS.B1713 Failed Passed
II-12254 logical.calcs Filter.TopN-context Failed Passed

Test Results

test_results_combined-BeforeFix.csv test_results_combined-AfterFix.csv

hab6 commented 11 months ago

The connector capability setting CAP_CREATE_TEMP_TABLES=no resolves each of the 6 tests listed in this PR.

Jira Ticket Test Name Change Note
II-12247 Query.Sort_Alphabetic_DESC_Top1 Temporary table clause is removed from the Generated SQL
II-12248 calcs.union Generated SQL is changed from 1 statement to 3 statements. This seems unrelated to the capability setting change, although allows the test to succeed. This might need to be looked at again in the future.
II-12251 lod.calc.1.TopN Temporary table clause is removed from the Generated SQL
II-12252 BUGS.B340 Temporary table clause is removed from the Generated SQL
II-12253 BUGS.B1713 Temporary table clause is removed from the Generated SQL
II-12254 Filter.TopN-context Temporary table clause is removed from the Generated SQL

Issue 1185 has been logged with the Tableau Connector SDK team to better understand the connector capability settings CAP_CREATE_TEMP_TABLES and CAP_SELECT_INTO.

Will proceed with merge of this change.