Skip to content

refactor: decouple read/write responsibilities and introduce ColumnBinding - #1128

Open
bengbengbalabalabeng wants to merge 8 commits into
apache:mainfrom
bengbengbalabalabeng:refactor-converters
Open

refactor: decouple read/write responsibilities and introduce ColumnBinding#1128
bengbengbalabalabeng wants to merge 8 commits into
apache:mainfrom
bengbengbalabalabeng:refactor-converters

Conversation

@bengbengbalabalabeng

@bengbengbalabalabeng bengbengbalabalabeng commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Purpose of the pull request

Related: #1104

What's changed?

  • Separate Converter into WriteConverter (write-specific) and ReadConverter (read-specific) while preserving backward compatibility
  • Introduce ColumnBinding to support column-scoped converters in dynamic/no-bean scenarios
  • Introduce hierarchical CellDataConverterRegistry supporting local-first delegation
  • Eliminate the tight coupling between CSV export and CellDataType in write workflows
  • Ensure template fill compatibility, supporting both native cell writes and multi-variable text interpolation

For more examples, please refer to: #1104

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

@bengbengbalabalabeng
refactor: redesign converter architecture with read-write separation …
64bd9d8 
…and column binding

- Separate Converter into WriteConverter (write-specific) and ReadConverter (read-specific)
- Introduce ColumnBinding to support column-scoped converters in dynamic/no-bean scenarios
- Introduce hierarchical CellDataConverterRegistry supporting local-first delegation
- Implement LIFO precedence for custom converters
- Eliminate the issue where CSV types strongly depend on CellDataType in writing
@bengbengbalabalabeng
refactor: simplified converter grouping
0af9999
@bengbengbalabalabeng
refactor: compatible with template filling behavior
e55bd58
@bengbengbalabalabeng
bengbengbalabalabeng marked this pull request as ready for review October 7, 2026 11:41
@Sean-Walker0 Sean-Walker0 mentioned this pull request Oct 8, 2026
Open
3 tasks done
@Mikkey-f Mikkey-f mentioned this pull request Oct 11, 2026
Open
This was referenced Oct 15, 2026
Open
Open

This branch has not been deployed

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bengbengbalabalabeng