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.

Sign up for GitHub

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

Jump to bottom

[RELEASE] cudf v22.12 #12200

Merged
merged 252 commits into from
Dec 8, 2022
Merged

[RELEASE] cudf v22.12 #12200

merged 252 commits into from
Dec 8, 2022

Conversation

GPUtester
Copy link
Collaborator

❄️ Code freeze for branch-22.12 and v22.12 release

What does this mean?

Only critical/hotfix level issues should be merged into branch-22.12 until release (merging of this PR).

What is the purpose of this PR?

  • Update documentation
  • Allow testing for the new release
  • Enable a means to merge branch-22.12 into main for the release

GPUtester and others added 30 commits September 23, 2022 17:32
@GPUtester
Merge pull request #11757 from rapidsai/branch-22.10
ba9c43c 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11758 from rapidsai/branch-22.10
7376f1f 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11763 from rapidsai/branch-22.10
59847c1 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11767 from rapidsai/branch-22.10
41474af 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11773 from rapidsai/branch-22.10
5fb657d 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11774 from rapidsai/branch-22.10
2b94483 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11775 from rapidsai/branch-22.10
7d40b30 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11776 from rapidsai/branch-22.10
a1cbb02 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11777 from rapidsai/branch-22.10
aa2ef0e 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11781 from rapidsai/branch-22.10
cc6f237 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11782 from rapidsai/branch-22.10
cc97584 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11784 from rapidsai/branch-22.10
1d7af9e 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11786 from rapidsai/branch-22.10
b8ab576 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@davidwendt
Merge branch-22.10 into branch 22.12
54480f3
@davidwendt
add change from 11771
f72c4ce
@raydouglass
Merge pull request #11801 from davidwendt/branch-22.12-merge-22.10
017d85f 
Merge branch-22.10 into branch-22.12
@GPUtester
Merge pull request #11805 from rapidsai/branch-22.10
479514e 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11806 from rapidsai/branch-22.10
5cf7fdf 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11809 from rapidsai/branch-22.10
97353fc 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11810 from rapidsai/branch-22.10
69a031c 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11820 from rapidsai/branch-22.10
90afe92 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@davidwendt
Fix compile warning from CUDF_FUNC_RANGE in a member function (#11798)
ec4cdd8 
Compile warning was introduced in #11652 in `bgzip_data_chunk_source.cu`. The warning can be seen here https://gpuci.gpuopenanalytics.com/job/rapidsai/job/gpuci/job/cudf/job/prb/job/cudf-cpu-cuda-build/CUDA=11.5/12417/consoleFull (search for `177-D`)
```
/cudf/cpp/src/io/text/bgzip_data_chunk_source.cu(362): warning #177-D: variable "nvtx3_range__" was declared but never referenced
```
The `nvtx3_range__` is part of the `CUDF_FUNC_RANGE()` macro. The warning is incorrect and likely a compiler bug. The workaround in this PR is to add `[[maybe_unused]]` to the variable declaration.

I was not able to create a small reproducer for compile bug filing.

Authors:
  - David Wendt (https://github.com/davidwendt)

Approvers:
  - Tobias Ribizel (https://github.com/upsj)
  - MithunR (https://github.com/mythrocks)

URL: #11798
@GPUtester
Merge pull request #11821 from rapidsai/branch-22.10
87d0387 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11823 from rapidsai/branch-22.10
0ecbaa1 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11829 from rapidsai/branch-22.10
59ce915 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11830 from rapidsai/branch-22.10
c8c9027 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11831 from rapidsai/branch-22.10
3c9f9cf 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11832 from rapidsai/branch-22.10
cb81ebc 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11839 from rapidsai/branch-22.10
71167d7 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@GPUtester
Merge pull request #11851 from rapidsai/branch-22.10
8df6dbf 
[gpuCI] Forward-merge branch-22.10 to branch-22.12 [skip gpuci]
@madsbk
Reduce number of tests marked spilling (#12197)
a2f69e4 
To save CI running time, this PR reduce the tests marked `spilling` drastically. 

An alternative to #12187

Authors:
   - Mads R. B. Kristensen (https://github.com/madsbk)

Approvers:
   - https://github.com/brandon-b-miller
   - GALI PREM SAGAR (https://github.com/galipremsagar)
@codecov
Copy link

codecov bot commented Nov 18, 2022 •
edited
Loading

Codecov Report

Base: 10.56% // Head: 88.25% // Increases project coverage by +77.69% 🎉

Coverage data is based on head (9cd9841) compared to base (41a20f6).
Patch has no changes to coverable lines.

❗ Current head 9cd9841 differs from pull request most recent head f471bcc. Consider uploading reports for the commit f471bcc to get more accurate results

Additional details and impacted files 
@@             Coverage Diff             @@
##             main   #12200       +/-   ##
===========================================
+ Coverage   10.56%   88.25%   +77.69%     
===========================================
  Files         116      137       +21     
  Lines       18677    22571     +3894     
===========================================
+ Hits         1974    19921    +17947     
+ Misses      16703     2650    -14053
Impacted Files Coverage Δ
python/custreamz/custreamz/kafka.py 29.16% <0.00%> (-0.63%) ⬇️
python/dask_cudf/dask_cudf/backends.py 85.17% <0.00%> (-0.55%) ⬇️
python/dask_cudf/dask_cudf/sorting.py 93.29% <0.00%> (-0.09%) ⬇️
python/cudf/cudf/_fuzz_testing/io.py 0.00% <0.00%> (ø)
python/cudf/cudf/_fuzz_testing/orc.py 0.00% <0.00%> (ø)
python/cudf/cudf/_fuzz_testing/utils.py 0.00% <0.00%> (ø)
python/cudf/cudf/_fuzz_testing/fuzzer.py 0.00% <0.00%> (ø)
python/cudf/cudf/_fuzz_testing/parquet.py 0.00% <0.00%> (ø)
python/dask_cudf/dask_cudf/io/tests/test_csv.py 100.00% <0.00%> (ø)
python/dask_cudf/dask_cudf/io/tests/test_orc.py 100.00% <0.00%> (ø)
... and 144 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

ttnghia and others added 6 commits November 18, 2022 14:13
@ttnghia
Implement JNI for chunked Parquet reader (#11961)
782fba3 
This adds JNI for chunked Parquet reader. It depends on the chunked Parquet reader implementation PR  (#11867).

Authors:
   - https://github.com/nvdbaranec
   - Nghia Truong (https://github.com/ttnghia)

Approvers:
   - MithunR (https://github.com/mythrocks)
   - Robert (Bobby) Evans (https://github.com/revans2)
@vuule
Merge branch 'branch-22.12' of https://github.com/rapidsai/cudf into …
c79c2d1 
…bug-write_orc-compressission
@vuule
comment
08c0c5a
@vuule
Merge branch 'branch-22.12' of https://github.com/rapidsai/cudf into …
9292b50 
…bug-read_orc-empty-map-column
@galipremsagar
Fix dask backend dispatch (#12203)
21ba312 
This PR fixes a failure being observed in `dask` upstream: dask/dask#9676

Authors:
   - GALI PREM SAGAR (https://github.com/galipremsagar)

Approvers:
   - Richard (Rick) Zamora (https://github.com/rjzamora)
@vuule
fix is_data_empty
a8afc75
@mythrocks
Copy link
Contributor

I'm 👍 since #11867 and #11961 were merged.

@mythrocks
Copy link
Contributor

I should probably mention that this issue might need resolution:
NVIDIA/cub#594

vuule and others added 14 commits November 21, 2022 14:31
@vuule
remove assert; separate empty stripe and level
e670c10
@davidwendt
Workaround for CUB segmented-sort bug with boolean keys
cd6dff3
@vuule
Merge branch 'branch-22.12' of https://github.com/rapidsai/cudf into …
6756b02 
…bug-read_orc-empty-map-column
@vuule
test
f15080f
@jolorunyomi
Merge pull request #12217 from davidwendt/bug-cub-segmented-sort
49f983d 
Workaround for CUB segmented-sort bug with boolean keys
@jolorunyomi
Merge pull request #12160 from vuule/bug-read_orc-empty-map-column
ed35f67 
Fix data corruption when reading ORC files with empty stripes
@vyasr
Make dask pinning looser (#12231)
0c60819 
* Make pinning >=.

* Temporarily reenable wheel builds to ensure that things work as expected.

* Skip cudf tests and make sure dask-cudf builds.

* Undo changes to wheels scripts.
@vyasr
Fix include line for io/numpy.
c83ff55
@AyodeAwe
Merge pull request #12250 from vyasr/fix/io_numpy_link
eb27104 
Fix include line for IO Cython modules
@galipremsagar
merge
fc2ec42
@galipremsagar
Pin to 2022.11.1
297911f
@galipremsagar
Merge branch 'pin_dask' of https://github.com/galipremsagar/cudf into…
cbdefb8 
… pin_dask
@AyodeAwe
Merge pull request #12165 from galipremsagar/pin_dask
9cd9841 
[REVIEW] Pin `dask` and `distributed` for release
@raydouglass
update changelog
f471bcc
@raydouglass raydouglass merged commit a7dcfdf into main Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mythrocks mythrocks mythrocks approved these changes

@karthikeyann karthikeyann karthikeyann approved these changes

@ttnghia ttnghia ttnghia approved these changes

@shwina shwina Awaiting requested review from shwina shwina is a code owner automatically assigned from rapidsai/cudf-python-codeowners

@skirui-source skirui-source Awaiting requested review from skirui-source skirui-source is a code owner automatically assigned from rapidsai/cudf-python-codeowners

@bdice bdice Awaiting requested review from bdice bdice is a code owner automatically assigned from rapidsai/cudf-cpp-codeowners

Assignees
No one assigned
Labels
CMake CMake build issue Java Affects Java cuDF API. libcudf Affects libcudf (C++/CUDA) code. Python Affects Python cuDF API.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.