tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Query - separate timing for planning and execution

Browse Source 
Reviewers: mislav.bradac, buda

Reviewed By: buda

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D393
This commit is contained in:
florijan 2017-05-23 11:28:41 +02:00
parent fbf028a438
commit 04670f7328
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/query/interpreter.hpp
View File

@ -41,6 +41,8 @@ void Interpret(const std::string &query, GraphDbAccessor &db_accessor,
// generate frame based on symbol table max_position // generate frame based on symbol table max_position
Frame frame(symbol_table.max_position()); Frame frame(symbol_table.max_position());
clock_t planning_end_time = clock();
std::vector<std::string> header; std::vector<std::string> header;
std::vector<Symbol> output_symbols(logical_plan->OutputSymbols(symbol_table)); std::vector<Symbol> output_symbols(logical_plan->OutputSymbols(symbol_table));
if (!output_symbols.empty()) { if (!output_symbols.empty()) {
@ -75,9 +77,17 @@ void Interpret(const std::string &query, GraphDbAccessor &db_accessor,
throw QueryRuntimeException("Unknown top level LogicalOp"); throw QueryRuntimeException("Unknown top level LogicalOp");
} }
clock_t end_time = clock(); clock_t execution_end_time = clock();
double time_second = double(end_time - start_time) / CLOCKS_PER_SEC;
summary["query_time_sec"] = TypedValue(time_second); // helper function for calculating time in seconds
auto time_second = [](clock_t start, clock_t end) {
return TypedValue(double(end - start) / CLOCKS_PER_SEC);
};
summary["query_planning_time"] = time_second(start_time, planning_end_time);
summary["query_plan_execution_time"] =
time_second(planning_end_time, execution_end_time);
//
// TODO set summary['type'] based on transaction metadata // TODO set summary['type'] based on transaction metadata
// the type can't be determined based only on top level LogicalOp // the type can't be determined based only on top level LogicalOp
// (for example MATCH DELETE RETURN will have Produce as it's top) // (for example MATCH DELETE RETURN will have Produce as it's top)