nsivabalan commented on PR #6753: URL: https://github.com/apache/hudi/pull/6753#issuecomment-1260408466
I triaged as to why payloadClassUpdate test in deltastreamer was succeeding w/ this patch. I was expecting it to fail. apparently payload class is a top level config in deltastreamer and hence its not passed via regular props. So, it bypasses the validation we have. for now, lets proceed w/ the patch. later we can decide if we really need to add validation for payload class update as well. also, have reverted m1 pom changes. once CI is green, we are good to land. -- 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]
