boneanxs commented on PR #9113:
URL: https://github.com/apache/hudi/pull/9113#issuecomment-1620905284

   You can still use dynamic partition, in this way:
   
   ```sql
   insert overwrite hudi_cow_pt_tbl partition(dt, hh) select 13, 'a13', 1100,  
'2021-12-09', '12'
   ```
   
   the main point is that do we consider `insert overwrite hudi_cow_pt_tbl 
select 13, 'a13', 1100,  '2021-12-09', '12'` is a dynamic partition writing? I 
think @leesf 's view makes sense,  
https://github.com/apache/hudi/pull/7365#issuecomment-1343707001
   
   > @nsivabalan hi, here are my two cents: insert overwrite xxx values(xx,xxx) 
has very clear semantics, it means overwrite the entire table, insert overwrite 
xx partition(xx) values(xx,xxx) means insert overwrite partitions, but hudi 
handles overwrite partitions for overwrite table, which is a definite bug and i 
do not think we need to introduce a new operation for it.
   
   Also, this change can also keep the consistent behavior with spark sql, 
`insert overwrite hudi_cow_pt_tbl select 13, 'a13', 1100,  '2021-12-09', '12'` 
will overwrite the whole table.


-- 
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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to