yihua commented on PR #12161:
URL: https://github.com/apache/hudi/pull/12161#issuecomment-2494745956

   > We have lots of tests for merge mode. This test is just to make sure 
configs are propagating correctly for streaming. Using a custom merger proves 
that the configs are actually being used. So I think adding more to this 
testing is redundant
   
   Understood, but this functional test tests the merge modes with structured 
streaming specifically.  So we should add the two other modes with smoke checks.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to