Skip to content

Commit 41352ab

Browse files
committed
ensure query metrics counters always are numbers
1 parent b6d696b commit 41352ab

File tree

1 file changed

+4
-12
lines changed

1 file changed

+4
-12
lines changed

ext/couchbase.cxx

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -3801,18 +3801,10 @@ cb_Backend_document_query(VALUE self, VALUE statement, VALUE options)
38013801
rb_hash_aset(metrics, rb_id2sym(rb_intern("execution_time")), resp.meta.metrics->execution_time.count());
38023802
rb_hash_aset(metrics, rb_id2sym(rb_intern("result_count")), ULL2NUM(resp.meta.metrics->result_count));
38033803
rb_hash_aset(metrics, rb_id2sym(rb_intern("result_size")), ULL2NUM(resp.meta.metrics->result_size));
3804-
if (resp.meta.metrics->sort_count > 0) {
3805-
rb_hash_aset(metrics, rb_id2sym(rb_intern("sort_count")), ULL2NUM(resp.meta.metrics->sort_count));
3806-
}
3807-
if (resp.meta.metrics->mutation_count > 0) {
3808-
rb_hash_aset(metrics, rb_id2sym(rb_intern("mutation_count")), ULL2NUM(resp.meta.metrics->mutation_count));
3809-
}
3810-
if (resp.meta.metrics->error_count > 0) {
3811-
rb_hash_aset(metrics, rb_id2sym(rb_intern("error_count")), ULL2NUM(resp.meta.metrics->error_count));
3812-
}
3813-
if (resp.meta.metrics->warning_count > 0) {
3814-
rb_hash_aset(metrics, rb_id2sym(rb_intern("warning_count")), ULL2NUM(resp.meta.metrics->warning_count));
3815-
}
3804+
rb_hash_aset(metrics, rb_id2sym(rb_intern("sort_count")), ULL2NUM(resp.meta.metrics->sort_count));
3805+
rb_hash_aset(metrics, rb_id2sym(rb_intern("mutation_count")), ULL2NUM(resp.meta.metrics->mutation_count));
3806+
rb_hash_aset(metrics, rb_id2sym(rb_intern("error_count")), ULL2NUM(resp.meta.metrics->error_count));
3807+
rb_hash_aset(metrics, rb_id2sym(rb_intern("warning_count")), ULL2NUM(resp.meta.metrics->warning_count));
38163808
}
38173809

38183810
return res;

0 commit comments

Comments
 (0)