Skip to content

Conversation

@zhengruifeng
Copy link
Contributor

What changes were proposed in this pull request?

Simplify the support of column type Any

Why are the changes needed?

I checked all the Any parameters, and all of them supports the Column type now.
but there are two kinds of implementations, I think the approach of array_append is much simpler, so I try to unify the implementations:

@_try_remote_functions
def array_append(col: "ColumnOrName", value: Any) -> Column:
    return _invoke_function_over_columns("array_append", col, lit(value))

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI

Was this patch authored or co-authored using generative AI tooling?

No

@zhengruifeng
Copy link
Contributor Author

merged to master

@zhengruifeng zhengruifeng deleted the py_func_any branch June 27, 2024 04:39
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.

2 participants