Skip to content

GH-1061: Add codegen classifier jar for arrow-vector.#1062

Open
lriggs wants to merge 5 commits intoapache:mainfrom
lriggs:codgenClassifier
Open

GH-1061: Add codegen classifier jar for arrow-vector.#1062
lriggs wants to merge 5 commits intoapache:mainfrom
lriggs:codgenClassifier

Conversation

@lriggs
Copy link
Contributor

@lriggs lriggs commented Mar 9, 2026

What's Changed

Add a new codegen classifier jar for arrow-vector that contains tdd and other template files.

Closes #1061 .

@lriggs lriggs changed the title GH-1061 Add codegen classifier jar for arrow-vector. GH-1061: Add codegen classifier jar for arrow-vector. Mar 9, 2026
@github-actions

This comment has been minimized.

@lidavidm
Copy link
Member

lidavidm commented Mar 9, 2026

Can you explain how these are useful?

@telemenar
Copy link

Can you explain how these are useful?

Arrow's type system has many types, and projects building operations across Arrow data (comparisons, casts, aggregations) benefit from generating type-specialized code rather than hand-writing implementations for each type.

The TDD files provide a machine-readable definition of Arrow's types that enables this. Including them in the distribution allows downstream projects to generate code that stays in sync as Arrow's type system evolves.

@lidavidm
Copy link
Member

Thanks! Can this rationale be documented in the code itself?

@lriggs
Copy link
Contributor Author

lriggs commented Mar 10, 2026

I added a comment about the usefulness of the codegen files.

@lidavidm
Copy link
Member

Ah, I would've expected it next to the build config, sorry for being unclear 😅

@jbonofre jbonofre added the enhancement PRs that add or improve features. label Mar 10, 2026
@jbonofre
Copy link
Member

Thanks for the details @telemenar and @lriggs .

Can we add a note in the docs folder, quickly documenting this ?

It's a good enhancement to me, we should give a bit of context for the users 😄

@jbonofre jbonofre added this to the 19.0.0 milestone Mar 10, 2026
@lriggs
Copy link
Contributor Author

lriggs commented Mar 10, 2026

Thanks. I added a few notes.

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

Labels

enhancement PRs that add or improve features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add new classifier jar for arrow-vector tdd files

4 participants