Vladislav,

It’s weird to implement setClientInfo without provide a way to see values (MySql Connector J only put values in a hashtable, I do not know what it is for !)

On Oracle, Sybase, SQLServer and DB2, these values are pushed in processes system view.

As much as possible, we want avoid to use one solution per database vendor or a feature with a performance cost.

But I agree, it's boring if it’s implementation can change in future.

Thanks.

Regards

Jérôme.

De : Vladislav Vaintroub [mailto:vvaintroub@gmail.com]
Envoyé : vendredi 31 mars 2017 12:37
À : jerome brauge; bar@mariadb.org
Cc : MariaDB Developers (maria-developers@lists.launchpad.net); Diego Dupin
Objet : Re: [Maria-developers] bb-10.2-compatibility - processlist

On 3/31/2017 12:16 PM, jerome brauge wrote:

Hi Vladislav,

As I understand is that this plugin allow user to see their own session variables.

Here, we can see these information for each session (for monitoring purpose, licensing control, …)

Is it correct ?

Yes, I think you're  correct and I'm not .

I was chatting to Diego, our JDBC expert, he does not seem to recall the reason session variables were used. I assume the implementation can change without any notification.
MySQL Connector/J  does not issue SET as I understand at the moment, does not talk to DB at all.

But for monitoring purposes, there is something in performance schema you might find useful https://dev.mysql.com/doc/refman/5.6/en/session-connect-attrs-table.html , most current drivers populate  this table  with similar information to Appname, client user, client hostname

Regards,

Jérôme.

De : Vladislav Vaintroub [mailto:vvaintroub@gmail.com]
Envoyé : vendredi 31 mars 2017 12:06
À : jerome brauge; bar@mariadb.org
Cc : MariaDB Developers (maria-developers@lists.launchpad.net)
Objet : Re: [Maria-developers] bb-10.2-compatibility - processlist

Hi Jerome, Bar

is user_variables plugin not enough for that ? https://mariadb.com/kb/en/mariadb/information-schema-user_variables-table/

It is only one driver that sets these user variables, so it might not be worth to extend processlist even more for general case, just for that.

On 3/31/2017 11:05 AM, jerome brauge wrote:

Hello Alexander,

Here is a patch to add information provided by JDBC (setClientInfo) in information_schema.processlist (ApplicationName, ClientUser, ClientHostname).

Do you think that it can be integrate ?

Regards,

Jérôme.

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : maria-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp