apache / datafusion-python

Apache DataFusion Python Bindings
https://datafusion.apache.org/python
Apache License 2.0
343 stars 69 forks source link

feat: add case function (#447) #448

Closed mesejo closed 1 year ago

mesejo commented 1 year ago

Which issue does this PR close?

Closes #447

What changes are included in this PR?

Besides exposing the case function, this PR also adds the PyCaseBuilder class (to parallel Rust's CaseBuilder)

Are there any user-facing changes?

Yes, but the documentation should be generated automatically

alamb commented 1 year ago

Looks like there is one failing CI check

jdye64 commented 1 year ago

Yes, @mesejo there is a check-style test failing if you don't mind fixing. If not I can make the commit to resolve

mesejo commented 1 year ago

Thanks to both of you @alamb and @jdye64 for the review 🙏. The check-style test should be fixed now!

alamb commented 1 year ago

Thanks again @mesejo