fallintoplace opened a new pull request, #1245: URL: https://github.com/apache/iceberg-go/pull/1245
Summary - return partially loaded positional-delete chunks when a concurrent delete-file read fails - keep successfully read chunks on the result channel even if another worker cancels the group - add checked-allocator regression coverage for partial failure cleanup Why If one positional-delete file is loaded and a later delete-file read fails, the scanner callers already try to release any partial result. readAllDeleteFiles was returning nil on error, so those loaded Arrow chunks could be lost instead of released. Testing - go test ./table -run 'TestReadAllDeleteFilesReturnsPartialDeletesOnError|TestReleasePerFilePosDeletes' -count=1\n- go test ./table -count=1\n- go run github.com/golangci/golangci-lint/v2/cmd/[email protected] run --timeout=10m ./table/... -- 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]
