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

Fix merge conflict with branch-21.10 #3748

Merged
merged 2 commits into from
Oct 5, 2021
Merged

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Oct 4, 2021

NOTE: This must be merged with a merge commit, not squashed!

Fixes the merge conflict identified in #3747. Conflict was in GpuCoalesceBatches where collect and total time metrics were removed in 21.12.

…tabricks] (NVIDIA#3719)

* Fix issues with AQE and DPP enabled on Spark 3.2

Signed-off-by: Jason Lowe <jlowe@nvidia.com>

* Add canonicalized parameter for 301db shim

* Fix double-close when batch contains multiple columns

* Fix HostColumnVector deserialization

* Advertise CPU sort and partitioning expressions to avoid AQE replanning issues

Signed-off-by: Jason Lowe <jlowe@nvidia.com>

* Fix IndexOutOfBoundsException errors from WindowFunctionSuite

* Update other shims

* Fix 311db build

* Fix 311db GpuWindowsInPandasExec and update optimized sort comment

* Specify GpuPartitioning type when expecting a GPU partitioning argument

* Add ability to override withNewChildren and fix BatchedByKey

* Fix 301db GpuWindowInPandasExec

* Use separate parameter list to pass CPU expressions

* Exclude CPU expressions from assertOnGpu checks

* Fix scalastyle on imports
@jlowe jlowe added the build Related to CI / CD or cleanly building label Oct 4, 2021
@jlowe jlowe added this to the Oct 4 - Oct 15 milestone Oct 4, 2021
@jlowe jlowe self-assigned this Oct 4, 2021
@jlowe
Copy link
Member Author

jlowe commented Oct 4, 2021

build

@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (branch-21.12@db0c2d5). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head e811543 differs from pull request most recent head 32cf237. Consider uploading reports for the commit 32cf237 to get more accurate results
Impacted file tree graph

@@               Coverage Diff               @@
##             branch-21.12    #3748   +/-   ##
===============================================
  Coverage                ?   47.97%           
  Complexity              ?     1742           
===============================================
  Files                   ?      231           
  Lines                   ?    29766           
  Branches                ?     4958           
===============================================
  Hits                    ?    14281           
  Misses                  ?    13321           
  Partials                ?     2164           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db0c2d5...32cf237. Read the comment docs.

@tgravescs
Copy link
Collaborator

build

@jlowe
Copy link
Member Author

jlowe commented Oct 5, 2021

CI failed due to known issue #3742. Merging this to restore auto-merge functionality.

@jlowe jlowe merged commit b0823e9 into NVIDIA:branch-21.12 Oct 5, 2021
@jlowe jlowe deleted the fix-merge branch October 5, 2021 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to CI / CD or cleanly building
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants