Skip to content

[flink] Support creating table with blob in flink sql.#6351

Merged
JingsongLi merged 9 commits intoapache:masterfrom
leaves12138:flink_blob
Sep 30, 2025
Merged

[flink] Support creating table with blob in flink sql.#6351
JingsongLi merged 9 commits intoapache:masterfrom
leaves12138:flink_blob

Conversation

@leaves12138
Copy link
Copy Markdown
Contributor

Purpose

Support creating table with blob type in flink sql.
Related PIP: https://cwiki.apache.org/confluence/display/PAIMON/PIP-35:+Introduce+Blob+to+store+multimodal+data

Tests

API and Format

Documentation

@JingsongLi JingsongLi closed this Sep 29, 2025
@JingsongLi JingsongLi reopened this Sep 29, 2025
+ "multiple option group separated by commas. "
+ "Now only 'external-paths' is supported.");

public static final ConfigOption<String> BLOB_FIELD =
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add this to CoreOptions? Spark will use this too.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

this.splits = new LinkedList<>();
this.pool = new Pool<>(1);
this.pool.add(new FileStoreRecordIterator());
this.pool.add(new FileStoreRecordIterator(blobField));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you just pass readType here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

Copy link
Copy Markdown
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@JingsongLi JingsongLi merged commit daf482e into apache:master Sep 30, 2025
25 checks passed
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