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.

Sign up for GitHub

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

Jump to bottom

Update star macro to use adapter.quote #205

Merged
merged 4 commits into from
Apr 23, 2020
Merged

Update star macro to use adapter.quote #205

merged 4 commits into from
Apr 23, 2020

Commits on Apr 7, 2020

  1. Add identifier for Spark 

    The default identifier using the double quotes is not working when using Spark as backend.
We should use backticks in this case.

Also, there was a typo in the name `identifer.sql` so I renamed it to `identifier.sql`
    Bas Beelen committed Apr 7, 2020
    Configuration menu
    Copy the full SHA
    4e6e8ea View commit details
    Browse the repository at this point in the history

Commits on Apr 16, 2020

  1. Use adapter.quote instead of adding spark__identifier

    Bas Beelen committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    f04debb View commit details
    Browse the repository at this point in the history

  2. Remove identifier.sql because it is not used elsewhere

    Bas Beelen committed Apr 16, 2020
    Configuration menu
    Copy the full SHA
    ba87605 View commit details
    Browse the repository at this point in the history

Commits on Apr 22, 2020

  1. Revert deletion of identifier macro. Add deprecation warning

    Bas Beelen committed Apr 22, 2020
    Configuration menu
    Copy the full SHA
    c95ed6b View commit details
    Browse the repository at this point in the history