Skip to content

API: add column-default model support - #260

Closed
cbb330 wants to merge 1 commit into
chbush/orc-id-binding-backportfrom
chbush/orc-column-default-api
Closed

API: add column-default model support#260
cbb330 wants to merge 1 commit into
chbush/orc-id-binding-backportfrom
chbush/orc-column-default-api

Conversation

@cbb330

@cbb330 cbb330 commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Backport the NestedField column-default model (initial-default / write-default) from Apache Iceberg #9502.
  • Add Expressions.lit and the SchemaParser JSON round-trip for default literals.
  • Pure API/Core change: no reader behavior changes on its own; consumed by the ORC fill PR that stacks on top.

Stack (bottom -> top)

  1. ORC: bind struct fields by Iceberg field id in generic and Spark 3.1 readers #256 - ORC id-binding backport (base 1.2.x)
  2. this PR - API column-default model (base chbush/orc-id-binding-backport)
  3. ORC initial-default fill (base chbush/orc-column-default-api)
  4. Spark 3.1 wiring (base chbush/orc-initial-default-orc-fill)

Testing Done

  • iceberg-api and iceberg-core suites pass
  • SchemaParser round-trip test added
  • API/Core RevAPI, checkstyle, and formatting pass

Made with Cursor

Backport the NestedField initial/write-default APIs, literal construction, and schema JSON round-trip needed to carry declared defaults into older readers.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cbb330

cbb330 commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator Author

Closing: this was based on the wrong branch (1.2.x, which is essentially vanilla apache 1.2.x). The correct target is the OpenHouse dev branch openhouse-1.2.0, which already has the id-bound ORC readers and the NestedField column-default API (#250) — so the id-binding and API backports here were redundant.

Rebuilt as a clean 2-PR stack on openhouse-1.2.0:

@cbb330 cbb330 closed this Jul 23, 2026
@cbb330
cbb330 deleted the chbush/orc-column-default-api branch July 23, 2026 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant