You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "fallintoplace (via GitHub)" <gi...@apache.org> on 2026/08/09 21:59:21 UTC
[PR] fix(manifest): reject invalid manifest entries [iceberg-go]
fallintoplace opened a new pull request, #1741:
URL: https://github.com/apache/iceberg-go/pull/1741
## What changed
Validate decoded manifest entries before inheritance or filtering:
- reject unknown entry status values
- reject data files in delete manifests
- reject delete files in data manifests
- reject unknown file content values
## Why
The writer validates entry status, but the reader currently accepts any decoded status. An unknown status is not treated as deleted, so it can continue through scan planning as a live entry. Content mismatches are accepted in the same path.
## Testing
- added low-level Avro fixtures for each invalid status and content case
- `go test ./...`
- `go vet ./...`
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] fix(manifest): reject invalid manifest entries [iceberg-go]
Posted by "fallintoplace (via GitHub)" <gi...@apache.org>.
fallintoplace closed pull request #1741: fix(manifest): reject invalid manifest entries
URL: https://github.com/apache/iceberg-go/pull/1741
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] fix(manifest): reject invalid manifest entries [iceberg-go]
Posted by "zeroshade (via GitHub)" <gi...@apache.org>.
zeroshade merged PR #1741:
URL: https://github.com/apache/iceberg-go/pull/1741
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org