shunping commented on PR #35501: URL: https://github.com/apache/beam/pull/35501#issuecomment-3026095474
I am trying to find out the root cause of why we need MySQLdb after testcontainers upgrade. I followed the failed test code(https://github.com/apache/beam/blob/623e998f48ea5de2953286065ad77957e8c1b9a6/sdks/python/apache_beam/io/external/xlang_jdbcio_it_test.py#L123-L140), extracted the following code block and ran it on version 3.7.1 and 4.10.0. ```python from testcontainers.mysql import MySqlContainer container = MySqlContainer() container.start() print(container.get_connection_url()) ``` Connection url from version 3.7.1 ``` mysql+pymysql://test:test@localhost:32768/test ``` Connection url from version 4.10.0 ``` mysql://test:test@localhost:32770/test ``` In version 3.7.1, it uses pymysql driver. In version 4.10.0, it does not specify the driver and I think it is default to MySQLdb. -- 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: github-unsubscr...@beam.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
