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

Support for DDL statements? #24

Open
joegrasse opened this issue Mar 11, 2019 · 4 comments
Open

Support for DDL statements? #24

joegrasse opened this issue Mar 11, 2019 · 4 comments
Labels

Comments

@joegrasse
Copy link

Based on pull request #10, it looks like DDL statements are supported. Is that correct?

@tejasmanohar
Copy link
Contributor

tejasmanohar commented Mar 11, 2019

DDL are supported, but their result/output is somewhat broken (though you likely don't need that, I never have) since we skip the first line of Athena results (for SELECTs, it contains the header). That PR would fix this but lacks tests.

Feel free to pick up on it!

@joegrasse
Copy link
Author

Ok, thanks.

@muroon
Copy link

muroon commented Jan 13, 2021

Recently, we released speee/go-athena.

We have already merged the PR related to this issue and can keep maintaining this library and review / merge your PRs.
In addition, we added the following new features.

  • Add Docker environment
  • Add reviewdog configuration for linting
  • Remove Result Header when executing DDL
  • Use workgroup
  • Update tests (make each package up to date)
  • Introduce Result Mode (Query result acquisition mode)

Especially, we think the Result Mode is a cool feature, and we summarized the feature here.
Please check and try to use it!

@bhavanki bhavanki added the bug label Sep 4, 2024
@bhavanki
Copy link
Contributor

bhavanki commented Sep 4, 2024

Summarizing work done since back in the day:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants