Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-28 Thread via GitHub
RussellSpitzer commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2442234431 Tests Passed and Merged Thanks @jbonofre for the PR and @manuzhang , @findepi and @Fokko for the review. -- This is an automated message from the Apache Git Service. To

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-28 Thread via GitHub
RussellSpitzer merged PR #11291: URL: https://github.com/apache/iceberg/pull/11291 -- 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...@ic

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-28 Thread via GitHub
Fokko commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2441911132 @jbonofre @RussellSpitzer Took the liberty of fixing this through the GitHub UI. Let's get this in 🚀 -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-28 Thread via GitHub
RussellSpitzer commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2441899036 @jbonofre This has conflicts, @findepi you have the ability to merge this if you are done reviewing :) -- This is an automated message from the Apache Git Service. To respond t

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-26 Thread via GitHub
jbonofre commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2439877711 @findepi conflict resolved. Thanks. -- 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

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-25 Thread via GitHub
findepi commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2437181487 @jbonofre there is a conflict, you may want to resolve -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-24 Thread via GitHub
jbonofre commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2434638953 @rdblue @RussellSpitzer as discussed together, this is the PR to remove the spring-boot dependency and use JDK HTTP server directly for mocking. -- This is an automated message from

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-24 Thread via GitHub
jbonofre commented on code in PR #11291: URL: https://github.com/apache/iceberg/pull/11291#discussion_r1814511669 ## aliyun/src/test/java/org/apache/iceberg/aliyun/TestUtility.java: ## @@ -65,7 +65,7 @@ public static AliyunOSSExtension initialize() { } else { LOG.inf

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-23 Thread via GitHub
manuzhang commented on code in PR #11291: URL: https://github.com/apache/iceberg/pull/11291#discussion_r1814144426 ## aliyun/src/test/java/org/apache/iceberg/aliyun/TestUtility.java: ## @@ -65,7 +65,7 @@ public static AliyunOSSExtension initialize() { } else { LOG.in

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-23 Thread via GitHub
jbonofre commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2432883972 @findepi @manuzhang I fixed the spotless. Sorry for the inconvenience. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-23 Thread via GitHub
jbonofre commented on PR #11291: URL: https://github.com/apache/iceberg/pull/11291#issuecomment-2432562122 I'm checking why one check failed. -- 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 s

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-16 Thread via GitHub
jbonofre commented on code in PR #11291: URL: https://github.com/apache/iceberg/pull/11291#discussion_r1803771650 ## aliyun/src/test/java/org/apache/iceberg/aliyun/oss/mock/AliyunOSSMockLocalStore.java: ## @@ -121,8 +116,9 @@ void deleteBucket(String bucketName) throws IOExcepti

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-14 Thread via GitHub
jbonofre commented on code in PR #11291: URL: https://github.com/apache/iceberg/pull/11291#discussion_r1799860746 ## aliyun/src/test/java/org/apache/iceberg/aliyun/oss/mock/AliyunOSSMockLocalStore.java: ## @@ -121,8 +116,9 @@ void deleteBucket(String bucketName) throws IOExcepti

Re: [PR] Aliyun: Remove spring-boot dependency [iceberg]

2024-10-12 Thread via GitHub
findepi commented on code in PR #11291: URL: https://github.com/apache/iceberg/pull/11291#discussion_r1797752955 ## aliyun/src/test/java/org/apache/iceberg/aliyun/oss/mock/AliyunOSSMockLocalStore.java: ## @@ -121,8 +116,9 @@ void deleteBucket(String bucketName) throws IOExceptio