-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Fix finishedQueries metric, add metrics reporting in GetMappedRange t… #9796
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
Conversation
Doxense CI Report for Windows 10
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Doxense CI Report for Windows 10
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
…est [release-7.1] (apple#9785) * Fix finishedQueries metric, add metrics reporting in GetMappedRange test * refactor to make format work * resolve comments * Fix more comments * Fix bugs and change running time of test
Doxense CI Report for Windows 10
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x
|
Result of foundationdb-pr on Linux CentOS 7
|
| int checkStorageQueueSeconds; | ||
| int queueMaxLength; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change both to double?
M1 CI found an error because of this:
GetMappedRange.actor.g.cpp
In file included from /Users/ec2-user/foundationdb/fdbserver/workloads/GetMappedRange.actor.cpp:22:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/MutationLogReader.actor.h:25:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/MutationLogReader.actor.h:30:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/FDBTypes.h:35:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/Status.h:24:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/JSONDoc.h:23:
In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_writer_template.h:13:
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:485:9: error: no matching function for call to 'get_value'
return internal_::get_value(*this, internal_::Type_to_type<T>());
^~~~~~~~~~~~~~~~~~~~
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/JSONDoc.h:268:17: note: in instantiation of function template specialization 'json_spirit::Value_impl<json_spirit::Config_map<std::string>>::get_value<long>' requested here
out = pLast->get_value<T>();
^
/Users/ec2-user/foundationdb/fdbserver/workloads/GetMappedRange.actor.cpp:484:12: note: in instantiation of function template specialization 'JSONDoc::get<long>' requested here
role.get("query_queue_max", queryQueueMax);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3399:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont5' requested here
loopDepth = a_body1loopBody1cont5(loopDepth);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3272:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont2' requested here
loopDepth = a_body1loopBody1cont2(loopDepth);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3316:15: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont1' requested here
loopDepth = a_body1loopBody1cont1(result, loopDepth);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3231:151: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1when1' requested here
if (__when_expr_0.isReady()) { if (__when_expr_0.isError()) return a_body1Catch1(__when_expr_0.getError(), std::max(0, loopDepth - 1)); else return a_body1loopBody1when1(__when_expr_0.get(), loopDepth); };
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3220:49: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1' requested here
while (loopDepth == oldLoopDepth) loopDepth = a_body1loopBody1(loopDepth);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3199:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopHead1' requested here
loopDepth = a_body1loopHead1(loopDepth);
^
/Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3720:9: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1' requested here
this->a_body1();
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:442:5: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<int>' for 2nd argument
int get_value(const Value& value, Type_to_type<int>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:447:9: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<int64_t>' for 2nd argument
int64_t get_value(const Value& value, Type_to_type<int64_t>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:452:10: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<uint64_t>' for 2nd argument
uint64_t get_value(const Value& value, Type_to_type<uint64_t>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:457:8: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<double>' for 2nd argument
double get_value(const Value& value, Type_to_type<double>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:462:29: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::String_type>' for 2nd argument
typename Value::String_type get_value(const Value& value, Type_to_type<typename Value::String_type>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:467:23: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::Array>' for 2nd argument
typename Value::Array get_value(const Value& value, Type_to_type<typename Value::Array>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:472:24: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::Object>' for 2nd argument
typename Value::Object get_value(const Value& value, Type_to_type<typename Value::Object>) {
^
/Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:477:6: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<bool>' for 2nd argument
bool get_value(const Value& value, Type_to_type<bool>) {
^
1 error generated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you change both to double?
M1 CI found an error because of this:
GetMappedRange.actor.g.cpp In file included from /Users/ec2-user/foundationdb/fdbserver/workloads/GetMappedRange.actor.cpp:22: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/MutationLogReader.actor.h:25: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/MutationLogReader.actor.h:30: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/FDBTypes.h:35: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/Status.h:24: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/JSONDoc.h:23: In file included from /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_writer_template.h:13: /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:485:9: error: no matching function for call to 'get_value' return internal_::get_value(*this, internal_::Type_to_type<T>()); ^~~~~~~~~~~~~~~~~~~~ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/JSONDoc.h:268:17: note: in instantiation of function template specialization 'json_spirit::Value_impl<json_spirit::Config_map<std::string>>::get_value<long>' requested here out = pLast->get_value<T>(); ^ /Users/ec2-user/foundationdb/fdbserver/workloads/GetMappedRange.actor.cpp:484:12: note: in instantiation of function template specialization 'JSONDoc::get<long>' requested here role.get("query_queue_max", queryQueueMax); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3399:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont5' requested here loopDepth = a_body1loopBody1cont5(loopDepth); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3272:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont2' requested here loopDepth = a_body1loopBody1cont2(loopDepth); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3316:15: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1cont1' requested here loopDepth = a_body1loopBody1cont1(result, loopDepth); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3231:151: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1when1' requested here if (__when_expr_0.isReady()) { if (__when_expr_0.isError()) return a_body1Catch1(__when_expr_0.getError(), std::max(0, loopDepth - 1)); else return a_body1loopBody1when1(__when_expr_0.get(), loopDepth); }; ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3220:49: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopBody1' requested here while (loopDepth == oldLoopDepth) loopDepth = a_body1loopBody1(loopDepth); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3199:16: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1loopHead1' requested here loopDepth = a_body1loopHead1(loopDepth); ^ /Users/ec2-user/foundationdb_build_output_macos_arm64/fdbserver/workloads/GetMappedRange.actor.g.cpp:3720:9: note: in instantiation of member function 'GetMappedRangeWorkload::ReportMetricActorState<GetMappedRangeWorkload::ReportMetricActor>::a_body1' requested here this->a_body1(); ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:442:5: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<int>' for 2nd argument int get_value(const Value& value, Type_to_type<int>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:447:9: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<int64_t>' for 2nd argument int64_t get_value(const Value& value, Type_to_type<int64_t>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:452:10: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<uint64_t>' for 2nd argument uint64_t get_value(const Value& value, Type_to_type<uint64_t>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:457:8: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<double>' for 2nd argument double get_value(const Value& value, Type_to_type<double>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:462:29: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::String_type>' for 2nd argument typename Value::String_type get_value(const Value& value, Type_to_type<typename Value::String_type>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:467:23: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::Array>' for 2nd argument typename Value::Array get_value(const Value& value, Type_to_type<typename Value::Array>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:472:24: note: candidate function [with Value = json_spirit::Value_impl<json_spirit::Config_map<std::string>>] not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<typename Value_impl<Config_map<string>>::Object>' for 2nd argument typename Value::Object get_value(const Value& value, Type_to_type<typename Value::Object>) { ^ /Users/ec2-user/foundationdb/fdbclient/include/fdbclient/json_spirit/json_spirit_value.h:477:6: note: candidate function template not viable: no known conversion from 'Type_to_type<long>' to 'Type_to_type<bool>' for 2nd argument bool get_value(const Value& value, Type_to_type<bool>) { ^ 1 error generated.
it seems the issue is at line role.get("query_queue_max", queryQueueMax);, but I can change them both to double after fixing the issue anyway
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Doxense CI Report for Windows 10
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr-macos on macOS Ventura 13.x
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x
|
Result of foundationdb-pr-macos on macOS Ventura 13.x
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
not related |
Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
Result of foundationdb-pr-clang-ide on Linux CentOS 7
|
Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x
|
|
all not related |
Result of foundationdb-pr-macos on macOS Ventura 13.x
|
Result of foundationdb-pr on Linux CentOS 7
|
Result of foundationdb-pr-clang on Linux CentOS 7
|
Result of foundationdb-pr-cluster-tests on Linux CentOS 7
|
apple#9796) * Fix finishedQueries metric, add metrics reporting in GetMappedRange test [release-7.1] (apple#9785) * Fix finishedQueries metric, add metrics reporting in GetMappedRange test * refactor to make format work * resolve comments * Fix more comments * Fix bugs and change running time of test * use double for options
#9796) (#9861) * Fix finishedQueries metric, add metrics reporting in GetMappedRange test [release-7.1] (#9785) * Fix finishedQueries metric, add metrics reporting in GetMappedRange test * refactor to make format work * resolve comments * Fix more comments * Fix bugs and change running time of test * use double for options
…est (#9785)
this PR includes #9785 and #9808
20230327-151859-haofu-c00466a3a157e11b
Code-Reviewer Section
The general pull request guidelines can be found here.
Please check each of the following things and check all boxes before accepting a PR.
For Release-Branches
If this PR is made against a release-branch, please also check the following:
release-branchormainif this is the youngest branch)