leaves12138 commented on PR #7975:
URL: https://github.com/apache/paimon/pull/7975#issuecomment-4586768084

   Thanks for the large CLI contribution. I did a high-level pass over the 
module structure, command coverage, parser/printing utilities, and the test 
set. Since this is a 9k+ line new module with many commands and public UX 
decisions, I am not comfortable approving it in this broad form yet. I would 
suggest splitting follow-ups or getting explicit agreement on the CLI scope/UX 
first, especially around overlap with existing Python/SQL entry points, command 
stability, packaging, and which operations should be included in the first 
commit.
   


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