Skip to content
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

Fix showTables for Backwards Compatibility with name Property in QuestDB #213

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

argshook
Copy link
Contributor

PR Description

This PR includes a fix for the showTables function to ensure backwards compatibility with the name property. The upcoming versions of QuestDB will change the column name from name to table_name for the tables() query.

The web console has already started using the table_name property, but it's crucial to maintain compatibility with older versions of QuestDB.

Please note that this backwards compatibility code can be removed once the new version of QuestDB is released.

Commit log:

  • fix showTables to be backwards compatible with name property
    Upcoming QuestdB versions will change the column name from name -> table_name for tables() query.

    The web console is already using table_name property, but it should be backwards compatible with older versions of QuestDB.

    Once the new version of QuestDB is released, we can remove this
    backwards compatible code.

Upcoming QuestdB versions will change the column name from `name` -> `table_name` for `tables()` query.

The web console is already using `table_name` property, but it should be backwards compatible with older versions of QuestDB.

Once the new version of QuestDB is released, we can remove this
backwards compatible code.
@argshook argshook self-assigned this Oct 24, 2023
Copy link
Contributor

@marregui marregui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marregui marregui merged commit 6182010 into main Oct 24, 2023
1 check passed
@marregui marregui deleted the web-console/backwards-compatible-table-name branch October 24, 2023 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants