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

Use glue APIs instead of information schema #44

Closed
2 tasks done
nicor88 opened this issue Nov 21, 2022 · 6 comments
Closed
2 tasks done

Use glue APIs instead of information schema #44

nicor88 opened this issue Nov 21, 2022 · 6 comments
Milestone

Comments

@nicor88
Copy link
Contributor

nicor88 commented Nov 21, 2022

Based on #4 (comment)

Implementation should be covered here northvolt/dbt-athena@49c7111

Using Glue API is better as faster and should allow multi-threading.

TODO

  • Use glue api instead of information schema
  • Use delete table from glue api - consider to do for Iceberg as well, as timeout lead to not having full deleted data. Before dropping pick the location from glue catalog.
@nicor88
Copy link
Contributor Author

nicor88 commented Nov 21, 2022

@jessedobbelaere FYI.

@chrischin478
Copy link
Contributor

@nicor88 I'll make an attempt at the Glue API delete

@Jrmyy
Copy link
Contributor

Jrmyy commented Apr 20, 2023

👋🏻 Hello @chrischin478
Have you been able to work on this issue ?
Otherwise, if don't have time, I can take since I developed the part of about table info 🙂 .

@nicor88 nicor88 added this to the v1.4.4 milestone Apr 20, 2023
@chrischin478
Copy link
Contributor

@Jrmyy still giving it a crack. When should this be completed by? I don't want to take too long with this.

@nicor88
Copy link
Contributor Author

nicor88 commented Apr 20, 2023

we are planning to close this for the 1.4.4 milestone, that should happen in the next week.

@chrischin478
Copy link
Contributor

@Jrmyy if I haven't made significant progress by end of day 4/23, will that give you enough time to complete the feature before the milestone?

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

No branches or pull requests

3 participants