luoyuxia commented on PR #3494: URL: https://github.com/apache/fluss/pull/3494#issuecomment-4884586243
> Hi @luoyuxia, > > The new CI run (28710725908) surfaced another NoNodeException failure (`/tabletservers/tables/<id>/laketable`) for partitioned tables — traced to `MetadataManager.createTable` skipping `registerTableAssignment` when `tableAssignment == null`. > > On second thought, I'm going to put the coordinator-side pre-check back in. Without it, this PR can't ship without bundling a MetadataManager change (And I'm not sure if this is reasonable). With it, this PR stays focused on #3483 and the partitioned-table parent-znode issue can be tracked separately. I'll restore `ensureTableNotDeleted` in `handleCommitLakeTableSnapshotV2`, keep the new V2 catch in `LakeTableHelper.registerLakeTableSnapshotV2` (so the ZK boundary still rejects late commits), and re-run CI. > > Thanks! Thanks for your patient. Will have another review -- 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]
