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

Fix wrong implicit cast in query dumper

Browse Source 
Summary: All integers are signed in openCypher.

Reviewers: buda, llugovic

Reviewed By: buda

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D2746
This commit is contained in:
Matej Ferencevic 2020-04-01 16:19:09 +02:00
parent d04993df67
commit 39deba3b9e
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/query/dump.cpp
View File

@ -78,7 +78,7 @@ void DumpPropertyValue(std::ostream *os, const storage::PropertyValue &value) {
void DumpProperties(
std::ostream *os, query::DbAccessor *dba,
const std::map<storage::PropertyId, storage::PropertyValue> &store,
std::optional<uint64_t> property_id = std::nullopt) {
std::optional<int64_t> property_id = std::nullopt) {
*os << "{";
if (property_id) {
*os << kInternalPropertyId << ": " << *property_id;
@ -131,8 +131,7 @@ void DumpVertex(std::ostream *os, query::DbAccessor *dba,
"Unexpected error when getting properties.");
}
}
DumpProperties(os, dba, *maybe_props,
std::optional<uint64_t>(vertex.CypherId()));
DumpProperties(os, dba, *maybe_props, vertex.CypherId());
*os << ");";
}