-
Notifications
You must be signed in to change notification settings - Fork 71
Add arrow datafusion python dependency #1085
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
Merged
ayushdg
merged 60 commits into
dask-contrib:main
from
jdye64:add_arrow_datafusion_python
May 4, 2023
Merged
Changes from all commits
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
14932ad
Introduce Arrow-DataFusion-Python, the new Apache Arrow community Pyt…
jdye64 6ccf087
Merge remote-tracking branch 'upstream/main' into add_arrow_datafusio…
jdye64 62d49bd
point to test upstream branch with custom build command
jdye64 cdc236b
update conda build recipe to include rust compiler and setuptools-rus…
jdye64 9604b3e
update cargo lock
jdye64 dece095
lower setuptools-rust version for a test
jdye64 a48e587
Merge branch 'main' into add_arrow_datafusion_python
jdye64 5c141b2
Add actions for Rust environment setup
jdye64 605fd92
fix workflow syntax issue
jdye64 ded1595
Add RUSTFMT which output
jdye64 291f117
Set rustfmt environment variable and validate it sticks
jdye64 5800478
Updated Cargo.lock
jdye64 b4ce6bd
Add RUSTFMT to the export script_env variables for the conda build
jdye64 e8e037e
Set default to stable
jdye64 b59a7d8
Merge branch 'arrow_datafusion_python_refactor' into add_arrow_datafu…
jdye64 35bc094
Point to typify version with rustfmt fix, and update conda build file
jdye64 4e6228b
Merge branch 'add_arrow_datafusion_python' of github.com:jdye64/dask-…
jdye64 82f7f97
Point to version of arrow-datafusion-python with the typify rustfmt fix
jdye64 3635939
Refactoring: not from arrow-datafusion-python but general refactoring…
jdye64 d41b3cd
Add setuptools-rust back to conda meta.yaml recipe from where I had a…
jdye64 38cfcdf
Remove rust install step and also don't use container amd64/rust
jdye64 0bf1cda
Add setup-builder back to conda workflow action
jdye64 c5ca4f0
Run apt-get commands as root
charlesbluca 1f0fc53
Add Rust installation to test.yml
jdye64 8951791
Add Rust install to Rust.yml file because protoc is now needed when r…
jdye64 9a57f73
Install protoc
jdye64 3d1e85e
tests
jdye64 fa18499
Merge remote-tracking branch 'upstream/main'
jdye64 cf3d9d4
merge with upstream/main
jdye64 482fc0a
Change github action files
jdye64 18e7a39
remove commented out code
jdye64 8bb5a30
Add protoc action to Rust CI stage
jdye64 eb394c3
Merge branch 'main' into add_arrow_datafusion_python
jdye64 1f2560f
Add protoc action to Rust CI stage
jdye64 cab8072
Add protoc to other CI build steps so that substrait support can now …
jdye64 bb2bc7b
Add protoc to conda workflow file
jdye64 3d6b2ce
Add Utf8 parsing support for WindowRanges
jdye64 3ded007
Get flat_name() from Column. This will include the table name in the …
jdye64 07aefac
Add protoc to github action step Test importing with bare requirements
jdye64 8f5b3d3
Fix cargo tests
jdye64 a984378
Add build.rs and target linker flags to ensure binary is linked again…
jdye64 cb81c8a
Merge remote-tracking branch 'upstream/main' into add_arrow_datafusio…
jdye64 24e67bf
Updates for testing
jdye64 febb742
Merge remote-tracking branch 'upstream/main' into add_arrow_datafusio…
jdye64 c74d944
Adjustments to fix failing pytests
jdye64 dc3a162
More Pytest fixes and Rust test fixes
jdye64 43efde1
Remove mistakenly left inplace breakpoint() calls
jdye64 6e8738a
Return np.timedelta instead of pd.timedelta for interval_month_day_nano
ayushdg 7b81fb0
protoc changes and small formatting review suggestions
jdye64 479e908
Merge branch 'add_arrow_datafusion_python' of github.com:jdye64/dask-…
jdye64 0415b0b
Include protoc back in rust.yml for cargo check command
jdye64 1d84460
Merge branch 'main' into add_arrow_datafusion_python
ayushdg 9deb9f0
set avg_days_in_month -> 30. Adjust update_depedencies script to poin…
jdye64 1689c5b
Merge branch 'main' into add_arrow_datafusion_python
jdye64 b7f1e72
Add back logic for avg_days_in_month
jdye64 04a638b
Merge remote-tracking branch 'origin/main' into add_arrow_datafusion_…
charlesbluca fe179dd
Set average days per month to 30
charlesbluca e1fd425
Adjust PyTest expected results to align with new avg_days_in_month of…
jdye64 9d76dcc
merge with upstream
jdye64 d08e4ed
Revert timestampdiff changes
charlesbluca File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,3 +32,4 @@ dependencies: | |
- tpot | ||
- tzlocal>=2.1 | ||
- uvicorn>=0.13.4 | ||
- libprotobuf=3 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,3 +31,4 @@ dependencies: | |
- tpot | ||
- tzlocal=2.1 | ||
- uvicorn=0.13.4 | ||
- libprotobuf=3 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,3 +32,4 @@ dependencies: | |
- tpot | ||
- tzlocal>=2.1 | ||
- uvicorn>=0.13.4 | ||
- libprotobuf=3 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
[target.x86_64-apple-darwin] | ||
jdye64 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
rustflags = [ | ||
"-C", "link-arg=-undefined", | ||
"-C", "link-arg=dynamic_lookup", | ||
] | ||
|
||
[target.aarch64-apple-darwin] | ||
rustflags = [ | ||
"-C", "link-arg=-undefined", | ||
"-C", "link-arg=dynamic_lookup", | ||
] |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.