Skip to content

Commit

Permalink
Fix materialized CTE plan issue
Browse files Browse the repository at this point in the history
  • Loading branch information
kryonix committed Apr 30, 2024
1 parent f6e169b commit b20894d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/include/duckdb/parser/query_node.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class CommonTableExpressionMap {
public:
CommonTableExpressionMap();

case_insensitive_map_t<unique_ptr<CommonTableExpressionInfo>> map;
std::map<string, unique_ptr<CommonTableExpressionInfo>> map;

public:
string ToString() const;
Expand Down
2 changes: 1 addition & 1 deletion src/include/duckdb/storage/serialization/nodes.json
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
{
"id": 100,
"name": "map",
"type": "case_insensitive_map_t<CommonTableExpressionInfo*>"
"type": "std::map<string, CommonTableExpressionInfo*>"
}
],
"pointer_type": "none"
Expand Down
4 changes: 2 additions & 2 deletions src/storage/serialization/serialize_nodes.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -272,12 +272,12 @@ unique_ptr<CommonTableExpressionInfo> CommonTableExpressionInfo::Deserialize(Des
}

void CommonTableExpressionMap::Serialize(Serializer &serializer) const {
serializer.WritePropertyWithDefault<case_insensitive_map_t<unique_ptr<CommonTableExpressionInfo>>>(100, "map", map);
serializer.WritePropertyWithDefault<std::map<string, unique_ptr<CommonTableExpressionInfo>>>(100, "map", map);
}

CommonTableExpressionMap CommonTableExpressionMap::Deserialize(Deserializer &deserializer) {
CommonTableExpressionMap result;
deserializer.ReadPropertyWithDefault<case_insensitive_map_t<unique_ptr<CommonTableExpressionInfo>>>(100, "map", result.map);
deserializer.ReadPropertyWithDefault<std::map<string, unique_ptr<CommonTableExpressionInfo>>>(100, "map", result.map);
return result;
}

Expand Down

0 comments on commit b20894d

Please sign in to comment.