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 1, 2024
1 parent b836e19 commit dd553cf
Show file tree
Hide file tree
Showing 6 changed files with 1,696 additions and 837 deletions.
Loading

0 comments on commit dd553cf

Please sign in to comment.