Re: [PR] HadoopFileIO to support bulk delete through the Hadoop Filesystem APIs [iceberg]

2025-01-23 Thread via GitHub
rdblue commented on code in PR #10233: URL: https://github.com/apache/iceberg/pull/10233#discussion_r1927789250 ## core/src/main/java/org/apache/iceberg/hadoop/wrappedio/BindingUtils.java: ## @@ -0,0 +1,202 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] HadoopFileIO to support bulk delete through the Hadoop Filesystem APIs [iceberg]

2025-01-23 Thread via GitHub
rdblue commented on code in PR #10233: URL: https://github.com/apache/iceberg/pull/10233#discussion_r1927789250 ## core/src/main/java/org/apache/iceberg/hadoop/wrappedio/BindingUtils.java: ## @@ -0,0 +1,202 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one +

Re: [PR] HadoopFileIO to support bulk delete through the Hadoop Filesystem APIs [iceberg]

2025-01-23 Thread via GitHub
steveloughran commented on PR #10233: URL: https://github.com/apache/iceberg/pull/10233#issuecomment-2610564022 Closing as is because now that Hadoop 3.4.1 is the hadoop version, this can be done without reflection. Doing that in a new PR; this one exists to show the reflection-based

Re: [PR] HadoopFileIO to support bulk delete through the Hadoop Filesystem APIs [iceberg]

2025-01-22 Thread via GitHub
steveloughran commented on code in PR #10233: URL: https://github.com/apache/iceberg/pull/10233#discussion_r1925882790 ## core/src/main/java/org/apache/iceberg/hadoop/HadoopFileIO.java: ## @@ -166,23 +178,106 @@ public void deletePrefix(String prefix) { @Override public vo