hackergin commented on PR #23133: URL: https://github.com/apache/flink/pull/23133#issuecomment-1666721407
Thank you for addressing this issue. Over looks good to me. Only one minor comment. Could we incorporate a test case into the `testCatalogStore`? This would involve verifying that after invoking the getCatalog method, the Catalog instance is properly stored in catalogs (initialized catalog). WDYT ? -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org