Skip to content

register datafusion.functions as a python package#995

Merged
houqp merged 1 commit into
apache:masterfrom
houqp:qp_pypackage
Sep 14, 2021
Merged

register datafusion.functions as a python package#995
houqp merged 1 commit into
apache:masterfrom
houqp:qp_pypackage

Conversation

@houqp
Copy link
Copy Markdown
Member

@houqp houqp commented Sep 13, 2021

Rationale for this change

Make it possible to import functions from functions package. Without this change, from datafusion.functions import col will result in Module not found error.

This is also needed to fix the import error reported by sphinx when building the python doc.

What changes are included in this PR?

register datafusion.functions as a python package

Are there any user-facing changes?

no

@houqp houqp added this to the 5.1.0 milestone Sep 13, 2021
@houqp
Copy link
Copy Markdown
Member Author

houqp commented Sep 13, 2021

cc @jimexist

@houqp houqp merged commit 2615628 into apache:master Sep 14, 2021
@houqp houqp deleted the qp_pypackage branch September 14, 2021 01:02
@houqp houqp added the bug Something isn't working label Nov 4, 2021
unkloud pushed a commit to unkloud/datafusion that referenced this pull request Mar 23, 2025
* update tuning guide

* update benchmarking guide

* more updates

* fix markdown syntax

* fix markdown syntax
HairstonE pushed a commit to HairstonE/datafusion that referenced this pull request Oct 7, 2025
* fix: correct LZ0 to LZO in compression options

* fix: disable LZO compression option and update tests to reflect its unavailability

* fix: ruff format expected string in test_execution_plan

* fix: update test for execution plan and add validation for invalid LZO compression

* fix: remove LZO compression option and related test cases

* ruff autoformat

* fix: remove TODO comment regarding LZO compression implementation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants