MySQL Bugs: #96794: Contribution: refactor(connection-url): reduces the use of reflection durin ...

Bug #96794	Contribution: refactor(connection-url): reduces the use of reflection durin ...
Submitted:	9 Sep 2019 16:28	Modified:	18 Jul 2020 10:35
Reporter:	OCA Admin (OCA)	Email Updates:
Status:	Won't fix	Impact on me:	None
Category:	Connector / J	Severity:	S4 (Feature request)
Version:		OS:	Any
Assigned to:		CPU Architecture:	Any

Description:
This bug tracks a contribution by Manyanda Chitimbo (Github user: machi1990) as described in http://github.com/mysql/mysql-connector-j/pull/43

How to repeat:
See description

Suggested fix:
See contribution code attached

Contribution submitted via Github - refactor(connection-url): reduces the use of reflection during instantiation of (*) Contribution by Manyanda Chitimbo (Github machi1990, mysql-connector-j/pull/43#issuecomment-529488132): I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.

Contribution: git_patch_315238578.txt (text/plain), 14.99 KiB.

Thank you for the contribution.

You are welcome. 

Hope this lands soon. I am working on making the usage of the mysql driver smooth in Quarkus.

Hi Manyanda,

Thanks for this contribution but I'm afraid this won't be possible. There are several places where the connector relies on reflection or dynamic proxies to support extensions and implement some of its features.

If I understand correctly, you are trying to remove all traces of reflection from the code. Why is this a problem for you and how it relates to Quarkus?

Hi Filipe,

The patch try to remove the usage of reflection wherever it is possible. This is to make the Mysql driver native compatible (using the GraalVM - https://www.graalvm.org/docs/reference-manual/native-image/). 

With this patch, the amount of classes (see https://github.com/machi1990/quarkus/blob/e2697d095a08a81eb75ec00bea5ef13e2af6625e/extensi...) needed to be registered for reflection (which are otherwise considered dead code and hence eliminated) is reduced considerably which in turn benefit application and frameworks developers looking to use the driver and GraalVM native image generation.

No feedback was provided for this bug for over a month, so it is
being suspended automatically. If you are able to provide the
information that was originally requested, please do so and change
the status of the bug back to "Open".