jerryshao commented on PR #13250: URL: https://github.com/apache/gravitino/pull/13250#issuecomment-5755164404
Hi @yuqi1129, thanks for the review. Two things from your summary: 1. **The API compatibility issue** (inline on `SupportsJobs.java:143`) — fixed at HEAD, replied inline with details. 2. **Multi-node correctness risk** (a non-owner node returning empty output indistinguishable from "no output produced") — you're right this is a real gap, and it's not new: it was raised earlier in this thread by @neoLsH and I said I'd address it as a follow-up rather than fold it into this PR, since it needs a proper design (request routing to the owning node vs. an explicit "not available on this node" signal vs. accepting it as a documented limitation) rather than a quick patch. That design discussion is in progress now. I'll link the follow-up issue/PR here once it's filed rather than block this PR on it, but happy to reconsider if you feel it should gate this one. _🤖 Addressed by [Claude Code](https://claude.com/claude-code)_ -- 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]
