Re: [PR] Core: Add ContentFileSet and ContentFileWrapper [iceberg]

2024-09-25 Thread via GitHub
aokolnychyi commented on code in PR #11195: URL: https://github.com/apache/iceberg/pull/11195#discussion_r1776041636 ## api/src/main/java/org/apache/iceberg/util/ContentFileSet.java: ## @@ -0,0 +1,212 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] Core: Add ContentFileSet and ContentFileWrapper [iceberg]

2024-09-25 Thread via GitHub
aokolnychyi commented on code in PR #11195: URL: https://github.com/apache/iceberg/pull/11195#discussion_r1776041636 ## api/src/main/java/org/apache/iceberg/util/ContentFileSet.java: ## @@ -0,0 +1,212 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] Core: Add ContentFileSet and ContentFileWrapper [iceberg]

2024-09-25 Thread via GitHub
aokolnychyi commented on code in PR #11195: URL: https://github.com/apache/iceberg/pull/11195#discussion_r1776041008 ## api/src/main/java/org/apache/iceberg/util/ContentFileSet.java: ## @@ -0,0 +1,212 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] Core: Add ContentFileSet and ContentFileWrapper [iceberg]

2024-09-24 Thread via GitHub
aokolnychyi commented on PR #11195: URL: https://github.com/apache/iceberg/pull/11195#issuecomment-2372638766 Will take a look tomorrow. Thanks, @nastra! -- 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