Add the getEstimatedRowSizeHistogram implementation
The getEstimatedRowSizeHistogram is based on the column_family.json that defined in the API. The implementation in the MBean and in the metrics is now works with the get histogram as long array method in the client. Signed-off-by: Amnon Heiman <amnon@cloudius-systems.com>
This commit is contained in:
parent
bde40aed53
commit
0ae282b548
@ -584,7 +584,7 @@ public class ColumnFamilyStore implements ColumnFamilyStoreMBean {
|
|||||||
@Deprecated
|
@Deprecated
|
||||||
public long[] getEstimatedRowSizeHistogram() {
|
public long[] getEstimatedRowSizeHistogram() {
|
||||||
log(" getEstimatedRowSizeHistogram()");
|
log(" getEstimatedRowSizeHistogram()");
|
||||||
return c.getLongArrValue("");
|
return c.getEstimatedHistogramAsLongArrValue("/column_family/metrics/estimated_row_size_histogram/" + getCFName());
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -225,7 +225,7 @@ public class ColumnFamilyMetrics {
|
|||||||
factory.createMetricName("EstimatedRowSizeHistogram"),
|
factory.createMetricName("EstimatedRowSizeHistogram"),
|
||||||
new Gauge<long[]>() {
|
new Gauge<long[]>() {
|
||||||
public long[] value() {
|
public long[] value() {
|
||||||
return c.getLongArrValue("/column_family/metrics/estimated_row_size_histogram/"
|
return c.getEstimatedHistogramAsLongArrValue("/column_family/metrics/estimated_row_size_histogram/"
|
||||||
+ cfName);
|
+ cfName);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@ -233,7 +233,7 @@ public class ColumnFamilyMetrics {
|
|||||||
factory.createMetricName("EstimatedColumnCountHistogram"),
|
factory.createMetricName("EstimatedColumnCountHistogram"),
|
||||||
new Gauge<long[]>() {
|
new Gauge<long[]>() {
|
||||||
public long[] value() {
|
public long[] value() {
|
||||||
return c.getLongArrValue("/column_family/metrics/estimated_column_count_histogram/"
|
return c.getEstimatedHistogramAsLongArrValue("/column_family/metrics/estimated_column_count_histogram/"
|
||||||
+ cfName);
|
+ cfName);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
Loading…
Reference in New Issue
Block a user