Skip to content

Conversation

@beliefer
Copy link
Contributor

What changes were proposed in this pull request?

#41608 add bit operations to Scala, Python and Connect API, but missing the test for DataFrameFunctionsSuite.

Why are the changes needed?

Fix missing tests

Does this PR introduce any user-facing change?

'No'.
New feature.

How was this patch tested?

N/A

@github-actions github-actions bot added the SQL label Jun 17, 2023
@beliefer
Copy link
Contributor Author

ping @zhengruifeng cc @LuciferYang

@LuciferYang
Copy link
Contributor

OK, let me close #41639

@beliefer
Copy link
Contributor Author

@LuciferYang Because the issue is my fault. Let me fix it. Thank you!

@beliefer beliefer changed the title [SPARK-43928][SQL][PYTHON][CONNECT][FOLLOWUP] Update excludedSqlFunctions of DataFrameFunctionsSuite [SPARK-43928][SQL][PYTHON][CONNECT][FOLLOWUP] Update excludedSqlFunctions of DataFrameFunctionsSuite Jun 17, 2023
@panbingkun
Copy link
Contributor

LGTM

@zhengruifeng
Copy link
Contributor

I manually check this fix in my local env

merged to master

@zhengruifeng
Copy link
Contributor

maybe we should move DataFrameFunctionsSuite outside of sql - other to avoid similar mistake, I will give a try

@beliefer
Copy link
Contributor Author

maybe we should move DataFrameFunctionsSuite outside of sql - other to avoid similar mistake, I will give a try

Yeah. The sql - other test always failed and let us confused.

@dongjoon-hyun
Copy link
Member

Thank you, @beliefer and all!

czxm pushed a commit to czxm/spark that referenced this pull request Jun 19, 2023
…tions` of `DataFrameFunctionsSuite`

### What changes were proposed in this pull request?
apache#41608 add bit operations to Scala, Python and Connect API, but missing the test for `DataFrameFunctionsSuite`.

### Why are the changes needed?
Fix missing tests

### Does this PR introduce _any_ user-facing change?
'No'.
New feature.

### How was this patch tested?
N/A

Closes apache#41640 from beliefer/SPARK-43928_followup.

Authored-by: Jiaan Geng <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants