Skip to content

Conversation

itholic
Copy link
Contributor

@itholic itholic commented Aug 18, 2021

What changes were proposed in this pull request?

This PR is followup for #33646 to add missing tests.

Why are the changes needed?

Some tests are missing

Does this PR introduce any user-facing change?

No

How was this patch tested?

Unittest

…lling and groupby-expanding to follow pandas 1.3
@itholic
Copy link
Contributor Author

itholic commented Aug 18, 2021

cc @ueshin

@SparkQA
Copy link

SparkQA commented Aug 18, 2021

Test build #142584 has finished for PR 33776 at commit 0f594a6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 18, 2021

Kubernetes integration test unable to build dist.

exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/47083/

Copy link
Member

@ueshin ueshin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@ueshin
Copy link
Member

ueshin commented Aug 18, 2021

Thanks! merging to master.

@ueshin ueshin closed this in c91ae54 Aug 18, 2021
HyukjinKwon pushed a commit that referenced this pull request Aug 27, 2021
…lling and groupby-expanding to follow pandas 1.3

This PR is followup for #33646 to add missing tests.

Some tests are missing

No

Unittest

Closes #33776 from itholic/SPARK-36388-followup.

Authored-by: itholic <haejoon.lee@databricks.com>
Signed-off-by: Takuya UESHIN <ueshin@databricks.com>
(cherry picked from commit c91ae54)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants