HuangFru comments

Results 4 comments of


                                            HuangFru

[AMORO-2365] Fix get target snapshot NPE when commiting

**When the table is planning:** ![image](https://github.com/NetEase/amoro/assets/68625618/d40305da-85b7-4c65-8f23-38a7a59ad461) ![image](https://github.com/NetEase/amoro/assets/68625618/6df2adb8-b7da-440b-9c72-cbf5cfbb0afa) **After planning:** ![image](https://github.com/NetEase/amoro/assets/68625618/622005f2-769c-4389-ba52-a212cf87a513) ![image](https://github.com/NetEase/amoro/assets/68625618/90ca83a2-eb12-4690-983c-6146d977fcdf) **When the table is planning:** ![image](https://github.com/NetEase/amoro/assets/68625618/2f4f2ead-08fe-4bd4-a445-5fd768ed31c0) ![image](https://github.com/NetEase/amoro/assets/68625618/47f1ad68-3629-4d72-ba7a-a7a55bee10b3) **Restart the AMS:** ![image](https://github.com/NetEase/amoro/assets/68625618/03e0c5f4-1f6b-420e-9deb-ea16325d5108) **When the table is planning I set `self-optimizing.enabled`...

[AMORO-2365] Fix get target snapshot NPE when commiting

> I left some comments, please take a look @HuangFru Fixed.

[Feature]: Support rewriting manifest files automatically for tables

I think the benefit of automatically rewriting the manifest is small: From the perspective of management of fragmented files, according to `org.apache.iceberg.ManifestMergeManager#mergeGroup`, Iceberg will merge the small manifests automatically when...

[Bug]:Unable to find delete file

Thanks for reporting this bug. Can you describe this problem in detail? Which format? What engines? How to reproduce? Can you show the complete exception stack? And the Amoro community...