mchades commented on PR #4589: URL: https://github.com/apache/gravitino/pull/4589#issuecomment-2297962917
> I suggest we merge several JDK versions and use JDK17 to run the GitHub CI pipeline. If we want to test all JDK versions, I think a nightly test is okay. It's more probable that the storage backend is incorrect than the JDK version. > > Concerning the embedded and deploy modes, I suggest keeping them as they are and both modes are useful. Thanks for your suggestions! Running tests with multiple JDK versions only in nightly tests seems a bit aggressive. I suggest running tests with multiple JDK versions first on the main branch, as it is easier to discover issues this way. Once there is a new push to the main branch, old tasks will be automatically canceled, so there is no need to worry about having too many jobs. Based on actual usage scenarios, basically no one will use the h2 backend in a production environment (deploy mode). If suitable backends for deploy mode such as PG are added later, they can also be included in the deployment testing. -- 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]
