su8y commented on PR #7662: URL: https://github.com/apache/gravitino/pull/7662#issuecomment-3060147362
> Before merging this, please check all the related codes to see if there is no similar issue. Besides, you need to add the UT to cover the fix. Thanks for your time @jerryshao. Sorry, I hadn't considered the Python client as it was outside the initial scope of the issue. I also realized that while unit tests exist for the code I've worked on, there aren't any specifically for encoding. I'll correct this and upload the changes soon. @justinmclean I've observed that, in addition to `alterModelVersion()`, other methods within `GenericModelCatalog.java` also lack UT verifying proper `RESTUtils.encoding()`. Should these areas be addressed concurrently? -- 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]
