tomtongue commented on PR #10039: URL: https://github.com/apache/iceberg/pull/10039#issuecomment-2019863607
> @tomtongue can you also please include the other files from `core` that I mentioned in [#10027 (comment)](https://github.com/apache/iceberg/pull/10027#issuecomment-2017686753)? From my review, the following 8 classes need to be migrated. I'm going to add some files to this PR (if all files should be added, please let me know). - [ ] `actions/` - [ ] `TestCommitService` - [ ] `TestSizeBasedRewriter` - [ ] `avro/` - [ ] `TestNameMappingWithAvroSchema` - [ ] `encryption/` - [ ] `TestGcmStreams` - [ ] `TestStandardKeyMetadataParser` - [ ] `io/` - [ ] `TestOutputFileFactory` - [ ] `mapping/` - [ ] `TestMappingUpdates` - [ ] `TestNameMapping` -- 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-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
