-
Notifications
You must be signed in to change notification settings - Fork 110
@W-14212439: Understanding CAP_SELECT_INTO and CAP_CREATE_TEMP_TABLES #1286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
set to 'no' which will prevent tableau from generating statements and it will use the alternative approach.
docs/capabilities.md
Outdated
@@ -30,7 +30,7 @@ Capability | Description | Default | Recommended | |||
CAP_CREATE_TEMP_TABLES | Set to 'yes' if Tableau can create temporary tables needed for certain complex or optimized queries. Set to 'no' if creating temporary tables is not supported. See also: CAP_SELECT_INTO. | – | **<span style="color:red">?</span>** | |||
CAP_INDEX_TEMP_TABLES | Set to 'yes' if data source supports creation of indexes on temp tables database | – | – | |||
CAP_QUERY_USE_TEMP_TABLE_NAMES_AS_SUBQUERY_ALIASES | Set to 'yes' if Tableau must use generated temporary table names for aliases of subqueries because they might end up implemented as temporary tables | no | no | |||
CAP_SELECT_INTO | Set to 'yes' if Tableau can create a table on the fly from the resultset of another query. See also: CAP_CREATE_TEMP_TABLES. | – | **<span style="color:red">?</span>** | |||
CAP_SELECT_INTO | Set to 'yes' if Tableau can create a table on the fly from the resultset of another query. Set to 'no' which will prevent tableau from generating statements and it will use the alternative approach. See also: CAP_CREATE_TEMP_TABLES. | – | **<span style="color:red">?</span>** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rather than updating this file I'd see about trying to clarify this in https://tableau.github.io/connector-plugin-sdk/docs/design#temporary-table-support
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @rosswbrown ,
I have updated the code changes accordingly to the review comment. Please review it once you have time.
When 'CAP_SELECT_INTO' is set to "no", which will prevent tableau from generating statements and it will use the alternative approach.
Reverted the changes as no longer required and modified to the respective source file.
Agreed with suggestion. Co-authored-by: Ross Brown <[email protected]>
set to 'no' which will prevent tableau from generating statements and it will use the alternative approach.
Things To Remember:
master
if your PR runs on the current version of Tableau.dev-2021.2
README.md
if the PR is a release of the SDK, Connector Packager, or TDVT.=====