Skip to content

Generalize application information API #556

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

dkropachev
Copy link
Collaborator

In PR #554 application API was implemented in the form of three additional Configuration attributes.

This PR to have one Configuration attribute of ApplicationInfo interface. Default implementation of which will supply same three fields. ApplicationInfo interface is done in such a way that allows to inject any application information into startup message.

@dkropachev dkropachev requested a review from Bouncheck June 11, 2025 04:27
@dkropachev dkropachev self-assigned this Jun 11, 2025
@dkropachev dkropachev added the enhancement New feature or request label Jun 11, 2025
In PR scylladb#554 application API was implemented in the form of three
additional Configuration attributes.

This PR to have one Configuration attribute of `ApplicationInfo` interface.
Default implementation of which will supply same three fields.
`ApplicationInfo` interface is done in such a way that allows to inject
any application information into startup message.
@dkropachev dkropachev force-pushed the dk/generalize-application-info-api branch from 16fb35e to 4bebb7a Compare June 27, 2025 16:36
@dkropachev dkropachev merged commit 904fb97 into scylladb:scylla-3.x Jun 27, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants