Skip to content

Rework API RowVectors.partitionBySpec to support all PartitionFunctionSpec types#615

Merged
zhztheplayer merged 7 commits intoboostscale:mainfrom
zhztheplayer:wip-partition-func
Apr 15, 2026
Merged

Rework API RowVectors.partitionBySpec to support all PartitionFunctionSpec types#615
zhztheplayer merged 7 commits intoboostscale:mainfrom
zhztheplayer:wip-partition-func

Conversation

@zhztheplayer
Copy link
Copy Markdown
Contributor

@zhztheplayer zhztheplayer commented Apr 14, 2026

Maps Velox's PartitionFunction facility to Velox4J Java API to support all PartitionFunctionSpec types in RowVectors.partitionBySpec calls, and to provide a PartitionFunction API in Java for more flexible partitioning invocations.

Breaking change:

  • class PartitionFunctionSpec is being moved from org.boostscale.velox4j.plan.partition to org.boostscale.velox4j.partition

@zhztheplayer zhztheplayer marked this pull request as draft April 14, 2026 17:17
@zhztheplayer zhztheplayer changed the title Refactor PartitionFunction / PartitionFunctionSpec Rework API RowVectors.partitionBySpec to support all PartitionFunctionSpec types Apr 15, 2026
@zhztheplayer zhztheplayer marked this pull request as ready for review April 15, 2026 03:52
@zhztheplayer
Copy link
Copy Markdown
Contributor Author

zhztheplayer commented Apr 15, 2026

@LantaoJin I am working on generalizing the fundamentals of RowVectors.partitionBySpec through this PR. There's a breaking API change about moving the package:

class PartitionFunctionSpec is being moved from org.boostscale.velox4j.plan.partition to org.boostscale.velox4j.partition

@LantaoJin
Copy link
Copy Markdown
Contributor

@LantaoJin I am working on generalizing the fundamentals of RowVectors.partitionBySpec through this PR. There's a breaking API change about moving the package:

class PartitionFunctionSpec is being moved from org.boostscale.velox4j.plan.partition to org.boostscale.velox4j.partition

Wonder when could the maven deployment be trigger? each merged PR or nightly?

@zhztheplayer
Copy link
Copy Markdown
Contributor Author

@LantaoJin do you mean the Maven snapshot package? It's now manually triggered. Do you use it?

@LantaoJin
Copy link
Copy Markdown
Contributor

@LantaoJin do you mean the Maven snapshot package? It's now manually triggered. Do you use it?

Sure, I use the maven SNAPSHOT jar, please trigger it for any new API added or break changes.

@zhztheplayer zhztheplayer enabled auto-merge (squash) April 15, 2026 08:13
@zhztheplayer zhztheplayer disabled auto-merge April 15, 2026 08:14
@zhztheplayer zhztheplayer enabled auto-merge (squash) April 15, 2026 08:14
@zhztheplayer zhztheplayer disabled auto-merge April 15, 2026 08:17
@zhztheplayer zhztheplayer merged commit ec3ab1b into boostscale:main Apr 15, 2026
5 checks passed
@zhztheplayer
Copy link
Copy Markdown
Contributor Author

@LantaoJin released. The full deployment history is at https://github.com/boostscale/velox4j/actions/workflows/pub-snapshot.yml.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants