-
Notifications
You must be signed in to change notification settings - Fork 915
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
Forward-merge branch-24.04 into branch-24.06 [skip ci] #15330
Conversation
This PR updates the script inputs in the relevant workflows from `build_script` and `test_script` to `script`. Depends on rapidsai/shared-workflows#191
FAILURE - Unable to forward-merge due to an error, manual merge is necessary. Do not use the IMPORTANT: When merging this PR, do not use the auto-merger (i.e. the |
Resolves #15310. Contributes to #15193 In addition, this PR adds pylibcudf.Column<-->pyarrow.Array interconversion as a benefit Authors: - Vyas Ramasubramani (https://github.com/vyasr) Approvers: - Matthew Roeschke (https://github.com/mroeschke) URL: #15325
This PR also introduces `std::out_of_range` to cudf's code base in cases where it is appropriate. Contributes to #12885 Resolves #15315 Contributes to #15162 Authors: - Vyas Ramasubramani (https://github.com/vyasr) Approvers: - Bradley Dice (https://github.com/bdice) URL: #15319
In the Parquet writer, `decimal64` stats were being treated like `decimal128` (i.e. written in network byte order), when they should be treated like an `int64_t`. This PR fixes that and adds tests of `decimal32` and `decimal64` statistics. Authors: - Ed Seidl (https://github.com/etseidl) - Vukasin Milovanovic (https://github.com/vuule) Approvers: - David Wendt (https://github.com/davidwendt) - MithunR (https://github.com/mythrocks) - Vukasin Milovanovic (https://github.com/vuule) URL: #15281
…15324) This PR addresses the inconsistency in processing single quotes within a quoted string in the single quote normalizer. In the current implementation, when we have an escaped single quote within a single quoted string, the normalizer removes the backslash escape on converting the string to double quotes. However, the normalizer retains the contents of double quoted strings as-is i.e. if there are escaped single quotes within a double quoted string, the backslash character is retained in the output. We address this inconsistency by removing the escape character for single quotes in all double quoted string in the output. Tackles #15303 to mimic Spark behavior. Authors: - Shruti Shivakumar (https://github.com/shrshi) Approvers: - Bradley Dice (https://github.com/bdice) - Elias Stehle (https://github.com/elstehle) - Vukasin Milovanovic (https://github.com/vuule) URL: #15324
…r_test.cu` (#15200) In `parquet_chunked_reader_test.cu`, when generating test data, there is an `offset` value that should increase at every iteration. It is for shifting the null positions of each column such that the generating table will not have all nulls in the same rows. Somehow, it was left unchanged across all iterations, thus we need to fix that. Authors: - Nghia Truong (https://github.com/ttnghia) - Karthikeyan (https://github.com/karthikeyann) Approvers: - https://github.com/nvdbaranec - Bradley Dice (https://github.com/bdice) - Karthikeyan (https://github.com/karthikeyann) URL: #15200
Forward-merge triggered by push to branch-24.04 that creates a PR to keep branch-24.06 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge. See forward-merger docs for more info.