[ https://issues.apache.org/jira/browse/CASSANDRA-16378?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17459440#comment-17459440 ]
Tibor Repasi edited comment on CASSANDRA-16378 at 12/15/21, 8:11 AM: --------------------------------------------------------------------- The additional information has lowered the threshold to attack my first contribution to Cassandra. I've added most of the necessary things as described without breaking any unit tests so far. Linked the PR. Got stuck with the unit test, which I'd appreciate some hints how I can inject driver attributes, application_name and application_version, into the session to assert the contents of the virtual table. Still a question if nodetool would need change to expose the data on the command line. was (Author: rtib): The additional information has lowered the threshold to attack my first contribution to Cassandra. I've added most of the necessary things as described without breaking any unit tests so far. Linked the PR. Got stuck with the unit test, which I'd appreciate some hints how I can inject driver attributes into the session to assert the contents of the virtual table. Still a question if nodetool would need change to expose the data on the command line. > Expose application_name and application_version in virtual table > system_views.clients > ------------------------------------------------------------------------------------- > > Key: CASSANDRA-16378 > URL: https://issues.apache.org/jira/browse/CASSANDRA-16378 > Project: Cassandra > Issue Type: Improvement > Components: Feature/Virtual Tables > Reporter: Tibor Repasi > Assignee: Tibor Repasi > Priority: Normal > Labels: AdventCalendar2021, gsoc2021, lhf, mentor > > Recent java-driver's > [com.datastax.oss.driver.api.core.session.SessionBuilder|https://docs.datastax.com/en/drivers/java/4.9/com/datastax/oss/driver/api/core/session/SessionBuilder.html] > respects properties > [ApplicationName|https://docs.datastax.com/en/drivers/java/4.9/com/datastax/oss/driver/api/core/session/SessionBuilder.html#withApplicationName-java.lang.String-] > and > [ApplicationVersion|https://docs.datastax.com/en/drivers/java/4.9/com/datastax/oss/driver/api/core/session/SessionBuilder.html#withApplicationVersion-java.lang.String-]. > It would be helpful to exposed this information via virtual table > {{system_views.clients}} and with {{nodetool clientstats}}. > +Additional information for newcomers:+ > The drivers can send as part of the {{STARTUP MESSAGE}} the > {{APPLICATION_NAME}} and {{APPLICATION_VERSION}} options. To new volatile > fields {{applicationName}} and {{applicationVersion}} need to be added to > {{ClientState}} in a similar way to {{driverName}} and {{driverVersion}}. > The {{APPLICATION_NAME}} and {{APPLICATION_VERSION}} optionsneed to be > retrieved in {{StartupMessage#execute}} and passed to the {{ClientState}}. > The new {{application_name}} and {{application_version}} columns need to be > added to the {{system_views.clients}} represented by the {{ClientsTable}} > class. The data then need to be retrieved from the {{ClientState}} through > {{ConnectedClient}}. > Some unit tests similat to {{SettingsTableTest}} should be added. -- This message was sent by Atlassian Jira (v8.20.1#820001) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org