maropu commented on pull request #30515:
URL: https://github.com/apache/spark/pull/30515#issuecomment-734621562


   > We should use an image where one of the most recent three releases of 
MariaDB is installed, this PR solves the issue.
   If the local image tagged with 10.5 actually refers to one of the non-latest 
ones (10.5.6 or 10.5.7 for now), the plugin is not for the latest one.
   
   Hm, I see. If it is difficult to follow the rapid release cycle, how about 
parameterizing the image name as the oracle integration test does so? IMO it 
would be nicer so that we could always run the test on a latest image:
   
https://github.com/apache/spark/blob/d082ad0abfe0bc26760626ae0ecb415a8d508a1f/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala#L63
   Then, we can add a document about how-to-run it like 
`OracleIntegrationSuite`:
   
https://github.com/apache/spark/blob/d082ad0abfe0bc26760626ae0ecb415a8d508a1f/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/OracleIntegrationSuite.scala#L36-L56


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to