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 polars.scan_parquet #1143

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Use polars.scan_parquet #1143

wants to merge 1 commit into from

Conversation

mrocklin
Copy link
Member

@mrocklin
Copy link
Member Author

This is still not great. Performance is somewhat slow and I'm getting errors I don't yet understand on query 2:

  File "/Users/mrocklin/workspace/benchmarks/tests/tpch/conftest.py", line 349, in _run
  File "/Users/mrocklin/workspace/benchmarks/tests/tpch/test_polars.py", line 152, in _
  File "/opt/coiled/env/lib/python3.11/site-packages/polars/utils/deprecation.py", line 96, in wrapper
    return function(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/coiled/env/lib/python3.11/site-packages/polars/lazyframe/frame.py", line 1788, in collect
    return wrap_df(ldf.collect())
                   ^^^^^^^^^^^^^

PanicException('not yet supported')
  File "/Users/mrocklin/workspace/benchmarks/tests/tpch/conftest.py", line 349, in _run
  File "/Users/mrocklin/workspace/benchmarks/tests/tpch/test_polars.py", line 115, in _
  File "/opt/coiled/env/lib/python3.11/site-packages/polars/utils/deprecation.py", line 96, in wrapper
    return function(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/coiled/env/lib/python3.11/site-packages/polars/lazyframe/frame.py", line 1788, in collect
    return wrap_df(ldf.collect())
                   ^^^^^^^^^^^^^

ShapeError('unable to append to a dataframe of width 4 with a dataframe of width 9')

@ritchie46 I'm going to add you to this Coiled team in case you want to look at these exceptions in context (and look at performance metrics). No obligation to use it though if you don't want to.

@mrocklin
Copy link
Member Author

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.

1 participant