tsungchih commented on PR #9132: URL: https://github.com/apache/gravitino/pull/9132#issuecomment-3538550379
@jerryshao @unknowntpo Could you please help with reviewing this PR when you are available? I would greatly appreciate any feedback from you. Thanks. I've evaluated different ways to resolve the issue of cyclic-import between `RelationalTable` and `DTOConverters`. The refactor of [this commit](https://github.com/apache/gravitino/pull/9132/commits/d530771464fdafd62e5e480c7855e2cb97cde33d) achieves the least change and is consistent with the implementation in Java client. I therefore decided to conduct the refactor. Sorry for not being aware of the cyclic-import issue in advance when implementing `DTOConverters`. -- 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]
