yadavay-amzn commented on PR #55858: URL: https://github.com/apache/spark/pull/55858#issuecomment-4795635251
Thanks @cloud-fan, glad the guard checks out. Addressed the round-2 items in 5e9b8d2: - WriteDelta coverage: new `DeltaBasedMergeNondeterministicSuite` (merge-on-read) mirroring the reproduce-then-verify. - Folded the two near-duplicate group-based tests into one `rand()`-based test (rows + no group filter). - Updated the PR description for real this time (the `command == MERGE` gating, the `RowLevelOperationRuntimeGroupFiltering` `cond.deterministic` guard and why, and the new suites). For the remaining item - non-determinism in a MERGE action assignment (`SET x = uuid()`) landing on `MergeRows` - I filed a follow-up: SPARK-57685. PTAL. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
