Jackeyzhe commented on PR #3494:
URL: https://github.com/apache/fluss/pull/3494#issuecomment-4883044819

   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!


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