Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-31 Thread via GitHub
Amar1404 commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-2029144280 @CTTY : So here In EMR the default conf is getting applied, but as per the document of hudi if we specify the ENV HUDI_DEFAULT_CONF is not getting applied due to the bug in code, which i

Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-18 Thread via GitHub
yihua commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-2004976726 @Amar1404 any update on the PR and @CTTY ’s suggestion? -- 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

Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-07 Thread via GitHub
hudi-bot commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-1983881230 ## CI report: * 9d96c9e8d1416544638bfec3e8e98a1a5b3018b7 Azure: [SUCCESS](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=22

Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-07 Thread via GitHub
hudi-bot commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-1983736188 ## CI report: * 9d96c9e8d1416544638bfec3e8e98a1a5b3018b7 Azure: [PENDING](https://dev.azure.com/apache-hudi-ci-org/785b6ef4-2f42-4a89-8f0e-5f0d7039a0cc/_build/results?buildId=22

Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-07 Thread via GitHub
yihua commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-1983708727 @CTTY would you like to review this? -- 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 speci

Re: [PR] [MINOR] Changing the Properties to Load From Both Default Path and Enviorment [hudi]

2024-03-07 Thread via GitHub
hudi-bot commented on PR #10835: URL: https://github.com/apache/hudi/pull/10835#issuecomment-1983634301 ## CI report: * 9d96c9e8d1416544638bfec3e8e98a1a5b3018b7 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run th