Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-11 Thread via GitHub
leesf merged PR #9837: URL: https://github.com/apache/hudi/pull/9837 -- 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:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
stream2000 closed pull request #9837: [HUDI-6925] Do not list all partitions for 'alter table drop partition' URL: https://github.com/apache/hudi/pull/9837 -- 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

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
boneanxs commented on code in PR #9837: URL: https://github.com/apache/hudi/pull/9837#discussion_r1353913717 ## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala: ## @@ -326,10 +326,18 @@ object HoodieSqlCommonUtils

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1754960348 ## CI report: * 9c57c211d9866f753b9b08416880082ff70480da Azure:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1754672065 ## CI report: * 8128d5cbc6075f2b2fd7a8b7bf5e940d6493a66f Azure:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1754588535 ## CI report: * 8128d5cbc6075f2b2fd7a8b7bf5e940d6493a66f Azure:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
stream2000 commented on code in PR #9837: URL: https://github.com/apache/hudi/pull/9837#discussion_r1351709451 ## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala: ## @@ -326,10 +326,18 @@ object HoodieSqlCommonUtils

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
stream2000 commented on code in PR #9837: URL: https://github.com/apache/hudi/pull/9837#discussion_r1351710154 ## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala: ## @@ -326,10 +326,18 @@ object HoodieSqlCommonUtils

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
stream2000 commented on code in PR #9837: URL: https://github.com/apache/hudi/pull/9837#discussion_r1351709451 ## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala: ## @@ -326,10 +326,18 @@ object HoodieSqlCommonUtils

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-10 Thread via GitHub
boneanxs commented on code in PR #9837: URL: https://github.com/apache/hudi/pull/9837#discussion_r1351640998 ## hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/spark/sql/hudi/HoodieSqlCommonUtils.scala: ## @@ -326,10 +326,18 @@ object HoodieSqlCommonUtils

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-09 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1753009296 ## CI report: * 8128d5cbc6075f2b2fd7a8b7bf5e940d6493a66f Azure:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-09 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1752817160 ## CI report: * 8128d5cbc6075f2b2fd7a8b7bf5e940d6493a66f Azure:

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-09 Thread via GitHub
hudi-bot commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1752757006 ## CI report: * 8128d5cbc6075f2b2fd7a8b7bf5e940d6493a66f UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the

Re: [PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-09 Thread via GitHub
stream2000 commented on PR #9837: URL: https://github.com/apache/hudi/pull/9837#issuecomment-1752676594 @leesf @boneanxs Hi, could you help review this PR? Thanks~ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[PR] [HUDI-6925] Do not list all partitions for 'alter table drop partition' [hudi]

2023-10-09 Thread via GitHub
stream2000 opened a new pull request, #9837: URL: https://github.com/apache/hudi/pull/9837 ### Change Logs Currently we will list all partitions paths when calling 'alter table drop partition' to check if the table is hive style partitions and URL encoded. Listing all partitions