steFaiz commented on PR #7921:
URL: https://github.com/apache/paimon/pull/7921#issuecomment-4514814989
> Lines 1134-1138 of FlinkCatalog.java (the original blobTypeFields method)
are missing the blobExternalStoreField, while the newly added Spark side
blobTypeFields in PR and the blobTypeFields in the alterTable path both contain
the blobExternalStoreField. This leads to inconsistency in the determination of
blob fields between Flink's table creation path (from CatalogTable) and the
Alter TABLE path. The original method should be fixed to make it consistent.
@JingsongLi Thanks for pointing this out. This inconsistency did exist in
the original blobTypeFields method, but it has been fixed in the latest commit:
Flink now includes blobExternalStorageField() in blobTypeFields, so the create-
table path and alter-table path are consistent with Spark. Would you like
to checkout the latest commit?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
