chanwonlee commented on PR #9066: URL: https://github.com/apache/gravitino/pull/9066#issuecomment-3515115790
> Didn't it display "NONE,0" before? Sorry for the confusion, this is my first open source contribution and I'm still learning the process. It is correct that the original code already outputs "none,0". I misunderstood the issue and would appreciate clarification on what needs to be fixed. Looking at the original code: ```java printInformation(distribution.strategy() + "," + distribution.number()); ``` This already outputs "NONE,0" when the distribution is NONE. Could you help me understand what the actual issue is? I see two possibilities: 1. If "NONE,0" is the correct output, then no code changes are needed 2. If we want to modify the output for NONE (e.g., to make it clearer), I'm not sure what format would be best while maintaining backward compatibility I'd appreciate your guidance on the expected behavior so I can make the right changes. -- 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]
