Add support for poetry build with wheel#2425
Merged
HonahX merged 1 commit intoapache:mainfrom Sep 8, 2025
Merged
Conversation
eric-maynard
approved these changes
Sep 8, 2025
github-project-automation
bot
moved this from PRs In Progress
to Ready to merge
in Basic Kanban Board
HonahX
approved these changes
Sep 8, 2025
github-project-automation
bot
moved this from Ready to merge
to Done
in Basic Kanban Board
snazy
added a commit
to snazy/polaris
that referenced
this pull request
Nov 20, 2025
* Update dependency io.netty:netty-codec-http2 to v4.2.6.Final (apache#2520) * Update dependency org.testcontainers:localstack to v1.21.3 (apache#2497) * Allow overriding createCatalog calls in integrations tests (apache#2516) This is mostly to add testing flexibility in downstream projects. * Allow `PolarisServerManager` implementations to define custom client headers (apache#2510) `PolarisServerManager` is a plugin point for downstream builds to define runtime env. for tests under `integration-tests`. This change allows more flexibility for test runtime environments by allowing injecting extra headers into test clients. * Fix deprecation warnings around RandomStringUtils (apache#2507) the `RandomStringUtils.random` method is deprecated since commons-lang 3.17.0 apache/commons-lang@69cb996 * Add support for poetry build with wheel (apache#2425) * Core: Clarify the purpose of REPLACE_NEW_LOCATION_PREFIX_WITH_CATALOG_DEFAULT_KEY (apache#2509) * Update gradle/actions digest to ed40850 (apache#2524) * Update dependency com.fasterxml.jackson:jackson-bom to v2.20.0 (apache#2470) * Update hadoop to v3.4.2 (apache#2466) * Last merged commit a1d1176 --------- Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Dmitri Bourlatchkov <[email protected]> Co-authored-by: Christopher Lambert <[email protected]> Co-authored-by: Yong Zheng <[email protected]> Co-authored-by: Yufei Gu <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Earlier I created #2049 to show how we can now use
poetry buildto create sdist in python:With recent merge of #2192, we move the openapi code generation into python. The problem with this is, now it is depends on openapi template file which is outside python directory and
generate_clients.pyis referring the the template one directory above. Thus, we are no longer able to provide sdist (we never provided as well...often time sdist is less preferred).Now another issue I noticed is the wheel files generated by current main doesn't include the actual code files generated by
poetry buildbut the sdist has it. Upon debugging, this seems to be due to the project layout we are using is not matching to what poetry suggests:"If your project structure differs from the standard one supported by poetry, you can specify the packages you want to include in the final distribution." (ref: https://python-poetry.org/docs/pyproject/#packages). Here is the sample content if we try to generate one:
Thus, I purpose the change in this PR to implicitly add those files. Here is the new sample output:
Also, this PR add support with Makefile so now we can use
make client-buildto create the new wheel file.