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

   ### Describe the issue this Pull Request addresses
   
   - Support bootstrapping of col stats for MOR table.
   - Fix clean operation with col stats. Even though stats are nullified, the 
records apparently were not deleted from the col stats partition.
   
   ### Summary and Changelog
   
   - Support bootstrapping of col stats for MOR table.
   - Fix clean operation with col stats. Even though stats are nullified, the 
records apparently were not deleted from the col stats partition.
   
   ### Impact
   
   We could enable col stats for MOR table at any given state.
   Ran into other issues along the way which I had to fix to get the patch 
ready.
   
   - DirectoryInfo was not accounting for files fetched from MDT. When a new 
MDT partition is initialized, to fetch file info, we poll MDT rather than doing 
FS based listing. This had some a bug and had to fix it.
   - When clean from data table is applied to MDT, we were nullifying the stats 
or marking it as deleted, but the record as such is not deleted from col stats 
partition and was lingering. Fixed the same in this patch.
   
   
   Tests covered:
   
   - bootstrapping of both COW and MOR table.
   - Covered both partitioned and non-partitioned table.
   - Ensure log files w/ delete block, partially failed log blocks and rollback 
blocks are accounted for in tests.
   - Added tests to validate clean does remove the entry from col stats for 
both table types and partition and non-partitioned table.
   
   ### Risk Level
   
   low
   
   ### Documentation Update
   
   <!-- Describe any necessary documentation update if there is any new 
feature, config, or user-facing change. If not, put "none".
   
   - The config description must be updated if new configs are added or the 
default value of the configs are changed.
   - Any new feature or user-facing change requires updating the Hudi website. 
Please follow the 
     [instruction](https://hudi.apache.org/contribute/developer-setup#website) 
to make changes to the website. -->
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Enough context is provided in the sections above
   - [ ] Adequate tests were added if applicable
   


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