Skip to content

Commit

Permalink
Handles polars upgrade to 1.0.0
Browse files Browse the repository at this point in the history
Decisions:
1. Not backwards compatible -- we're effectively wrapping the library
   and we don't want to provide a bridge. This is polars versioning, not
   Hamilton versioning, however, so it's OK to keep this a patch bump
   (any polars code would have broken already).
2. They deprecated the usage of a connection string (sqllite://) in the
   PolarsDatabase{reader/writer} -- we've deprecated/changed the type as
   well
3. It switches based on polars version so people will only get the error
   for new polars upgrades

Otherwise, longer term, we're going to want to fix this -- moving away
from these blind wrappers.
  • Loading branch information
elijahbenizzy committed Jul 2, 2024
1 parent a12cb2f commit e3772dd
Show file tree
Hide file tree
Showing 6 changed files with 1,686 additions and 838 deletions.
Loading

0 comments on commit e3772dd

Please sign in to comment.