vlad-lyutenko commented on PR #13070:
URL: https://github.com/apache/iceberg/pull/13070#issuecomment-3265646003
```
Maybe you can suggest some way how to use
AvroIterable -> ValueReaders as pure reader, without this functionality.
```
To be more specific, when we use ` AvroIterable -> ValueReaders` to just
read data file we don't know `first_row_id` from metadata, so
`Long firstRowId = (Long) idToConstant.get(fieldId);`
doesn't contains this information and we always end up with
```
} else {
return ValueReaders.constant(null);
}
```
Instead of :
`return new RowIdReader(baseRowId, (ValueReader<Long>) idReader);`
which could actually read row_id from file.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
