Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

upgrade datafusion and pyo3 #20

Merged
merged 1 commit into from
Feb 22, 2022
Merged

upgrade datafusion and pyo3 #20

merged 1 commit into from
Feb 22, 2022

Conversation

jimexist
Copy link
Contributor

@jimexist jimexist commented Jan 26, 2022

need to wait for next release of datafusion

@jimexist jimexist marked this pull request as ready for review February 19, 2022 12:41
@matthewmturner
Copy link
Contributor

@jimexist Can I test this by using the instructions in How To Develop section of readme?

@matthewmturner
Copy link
Contributor

I ran tests on this branch and they passed

@matthewmturner
Copy link
Contributor

I'm trying to use this branch to run updated benchmarks for db-benchmark

when i use maturin develop im able to use it locally as expected.

however, when i use maturin develop --release I get the following when trying to import datafusion

Python 3.8.9 (default, Oct 26 2021, 07:25:53)
[Clang 13.0.0 (clang-1300.0.29.30)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import datafusion
Killed: 9

Any thoughts on what could cause this? Im looking into it on my side as well but wanted to raise it here in case anyone was familiar with this.

@matthewmturner
Copy link
Contributor

When I build the wheels with maturin build --release and install the wheel file directly it works. I guess you can ignore the above.

@jimexist jimexist merged commit 6c1ebe9 into main Feb 22, 2022
@jimexist jimexist deleted the datafusion-upgrade branch February 22, 2022 10:33
@jimexist
Copy link
Contributor Author

i sometimes will run into kill 9 by OS on my Mac - not sure if it's related but it's not persistent

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants