rangareddy commented on issue #16832:
URL: https://github.com/apache/hudi/issues/16832#issuecomment-5391416317

   This issue was reviewed as part of the JIRA-migrated backlog triage 
(HUDI-8987).
   
   **Findings: the gate exists; whether it actually blocks needs the test run.**
   
   The config is real and is consulted. `ENABLE_MERGE_INTO_PARTIAL_UPDATES` is 
declared at 
`hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/DataSourceOptions.scala:692`
 and read in `MergeIntoHoodieTableCommand.scala` at `:523-524`, with related 
handling at `:431` and an error message referencing it at `:1000`. (Note that 
is the `hudi-spark3-common` copy of the command - worth checking whether the 
Spark 4 path has the same handling.)
   
   So this is not a missing config; it is a claim that the gate does not take 
effect. Settling it means running the test named in the description ("Test 
merge into Allowed-patterns of assignment clauses") with the flag set to 
`false` on current `master` and confirming whether it still passes.
   
   If it does still pass, the interesting question is whether the flag is 
checked on the analysis path at all or only used to shape the write - which 
determines where the fix goes.
   


-- 
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