lasdf1234 commented on PR #11845: URL: https://github.com/apache/gravitino/pull/11845#issuecomment-5052779039
> > @Yohahaha There is only one minor point that needs to be changed. Could you add unit test about this feature? All the other issues have been solved. > > Thanks for the review! @lasdf1234 I think current testPaimonCompactProcedure already covers this new functionality. In the connector module of Flink, there is a unit test named TestGravitinoPaimonCatalog. Saprk would also be well-advised to have such an UT in place. Even if there are additional unique logics added later, this approach can still be adopted. -- 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]
