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

feat(impala): add tbl_properties to create_table #9839

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

jcrist
Copy link
Member

@jcrist jcrist commented Aug 14, 2024

TBLPROPERTIES can be set on table creation. Exposing this lets us un-xfail a few tests by setting the required TBLPROPERTIES to make them work.

@jcrist jcrist added the impala The Apache Impala backend label Aug 14, 2024
@cpcloud cpcloud merged commit e3d02bd into ibis-project:main Aug 14, 2024
84 checks passed
@jcrist jcrist deleted the impala-create-table branch August 14, 2024 18:35
gforsyth added a commit that referenced this pull request Dec 5, 2024
This removes the `ImpalaTable` subclass. Relevant methods that were on
that class have been moved to the `Backend` class instead.

We used to have backend-specific subclasses of `Table` that added some
backend-specific methods. Most of those were removed in prior releases,
`ImpalaTable` was the last remaining one.

As this is a breaking change, we should hold off on merging this until
we're ready to start prepping for a 10.0 release. Depends on #9839.

Fixes #9365.

---------

Co-authored-by: Gil Forsyth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impala The Apache Impala backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants