Skip to content
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.

Sign up for GitHub

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

Jump to bottom

FATE metrics should also include USER transactions #4534

Open
kevinrr888 opened this issue May 8, 2024 · 0 comments
Open

FATE metrics should also include USER transactions #4534

kevinrr888 opened this issue May 8, 2024 · 0 comments
Labels
enhancement This issue describes a new feature, improvement, or optimization.

Comments

@kevinrr888
Copy link
Contributor

Is your feature request related to a problem? Please describe.
FATE metrics currently only include META transactions (see: List<AdminUtil.TransactionStatus> currFates = admin.getTransactionStatus(Map.of(FateInstanceType.META, metaFateStore), null, null, null); in FateMetricValues). These metrics should also include USER transactions.

This was likely just something that was missed with the creation of UserFateStore in elasticity, since in 3.x and 2.x, UserFateStore doesn't exist.

Describe the solution you'd like
The FATE metrics should include all transactions (META and USER)

Additional context
This is for elasticity
See comments on #4498
@kevinrr888 kevinrr888 added the enhancement This issue describes a new feature, improvement, or optimization. label May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This issue describes a new feature, improvement, or optimization.
Projects
Accumulo Elasticity
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kevinrr888