anuragmantri commented on PR #57227:
URL: https://github.com/apache/spark/pull/57227#issuecomment-4960543758

   There is one case I'm not really sure about, I would like to know what you 
all think. There could be self-merge (same with self-insert) where relation 
cache will load the `t` with target options only and does not load the source 
options.
   
   ```sql
    MERGE INTO t WITH (`write.split-size`=10) USING t WITH (`split-size`=5) s 
ON ...
   ```
   
   `split-size` will be lost in this case for source read. I don't think this a 
common case to self merge. So I have not addressed it in this PR. Do you think 
we should address this? 


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

Reply via email to