[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-05-12 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1546422024 @hudi-bot run azure -- 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 un

[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-05-12 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1545726848 @bvaradar CI is green, could you please take a look at it again? thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-05-10 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1542814478 @hudi-bot run azure -- 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 un

[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-02-23 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1442298082 Hi Hudi devs, I would appreciate a review, 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

[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-02-23 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1441751764 There seem to be a bug with non-strict insert mode when using spark datasource it can insert duplicates only in overwrite mode or append mode when data is inserted to the table for the first

[GitHub] [hudi] kazdy commented on pull request #7998: [HUDI-5824] Fix: do not combine if write operation is Upsert and COMBINE_BEFORE_UPSERT is false

2023-02-21 Thread via GitHub
kazdy commented on PR #7998: URL: https://github.com/apache/hudi/pull/7998#issuecomment-1438816953 GH actions tests failed but I don't see why, it passed before. Azure failed on TestHoodieDeltaStreamerWithMultiWriter, rather unrelated and it also passed it on the first run. Looks like som