scxwhite opened a new pull request, #7236:
URL: https://github.com/apache/hudi/pull/7236

   ### Change Logs
   We put the public configuration in the file 
/etc/hudi/conf/hudi-defaults.conf,When the optimistic lock is opened, `alter 
table add column`  command will cause npe.
   
   
![image](https://user-images.githubusercontent.com/23207189/202611981-784680e5-1b33-42a3-9afb-74a13807b222.png)
   
   Then I found that because BaseHoodieWriteClient # preWrite was not executed 
in the AlterHoodieTableAddColumnsCommand class, 
pendingInflightAndRequestedInstants was not initialized. So submit this pr to 
fix it.
   <img width="1380" alt="image" 
src="https://user-images.githubusercontent.com/23207189/202612457-cb94c86a-4ae0-4a89-a124-0faae0dccb1a.png";>
   
   ### Impact
   
   alter table add/change column with optimistic lock is opened.
   ### Risk level (write none, low medium or high below)
   
   low
   
   ### Documentation Update
   
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


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