Stephen0421 commented on code in PR #9148:
URL: https://github.com/apache/paimon/pull/9148#discussion_r3790819093
##########
paimon-python/pypaimon/read/reader/blob_descriptor_convert_reader.py:
##########
@@ -239,5 +245,15 @@ def _normalize_blob_to_bytes(value):
value = bytes(value)
return value
+ @staticmethod
+ def _descriptor_field_to_blob(value, file_io):
+ if value is None:
+ return None
+ return Blob.from_descriptor_bytes(
+ value,
+ file_io=file_io,
+ uri_reader_factory=getattr(file_io, 'uri_reader_factory', None),
Review Comment:
Agreed. Descriptor materialization now uses `UriReaderFactory.from_file_io`
(HTTP(S) → `HttpUriReader`, other URIs → the supplied table FileIO).
`OffsetRow.get_blob()` and `BlobInlineConvertReader` no longer consult
`file_io.uri_reader_factory`. Added a token-aware FileIO regression that fails
if the catalog factory is used.
--
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]
