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

DBT-556 Added support for materializing Kudu table through impala adapter #207

Merged
merged 4 commits into from
Oct 22, 2024

Conversation

niranjancdw
Copy link
Collaborator

Describe your changes

Added support for materialising Kudu table through impala adapter.

Internal Jira ticket number or external issue link

https://jira.cloudera.com/browse/DBT-556

Testing procedure/screenshots(if appropriate):

https://gist.github.com/niranjancdw/cffbbd3dbe8126b9c96c25f645644c3d

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have formatted my added/modified code to follow pep-8 standards
  • I have checked suggestions from python linter to make sure code is of good quality.

@niranjancdw niranjancdw requested a review from niteshy October 8, 2024 13:10
@niranjancdw niranjancdw self-assigned this Oct 8, 2024
@kennycontreras
Copy link

I think in that case this one could be skipped then? #200

@niranjancdw
Copy link
Collaborator Author

I think in that case this one could be skipped then? #200

Yes, you can close that. We'll get this PR merged soon.

KUDU_INTEGRATION.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@niteshy niteshy left a comment

Choose a reason for hiding this comment

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

Let's merge this. Create a follow-up ticket to handle the partition tables.

@niranjancdw niranjancdw merged commit df1d2d9 into master Oct 22, 2024
@niranjancdw niranjancdw deleted the feature/kudu-support branch October 22, 2024 05:07
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