FANNG1 commented on PR #11060: URL: https://github.com/apache/gravitino/pull/11060#issuecomment-4457016930
Thanks, that makes sense to me regarding the default CI path only exercising one known-good version. One follow-up on the matrix runner itself: would you mind pinning the default `lance-namespace` version there as well, instead of resolving `latest`? Since the normal CI path is intentionally single-version and reproducible, it would be good if the matrix driver were reproducible by default too, rather than drifting with PyPI. Also, what do you think about the PR scope? The dependency upgrade / API adaptation part and the compatibility-matrix tooling/docs feel somewhat separable. I am okay either way, but if more CI wiring is planned later, splitting those concerns might make review and future maintenance a bit easier. -- 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]
