Open vikramsubramanian opened 4 months ago
Summary: Error message for mixing RDFGraph and regular property graph tables needs to be updated.
To address the issue described, you should update the error message in the code that throws the BinderException
when a relationship pattern mixes RDFGraph and regular Property graph tables. Based on the provided snippets, the relevant code is likely in the file src/binder/bind/bind_graph_pattern.cpp
.
src/binder/bind/bind_graph_pattern.cpp
that throws a BinderException
when a relationship pattern contains mixed graph types.The updated code snippet should look something like this:
if (!resourceTableIDs.empty()) {
if (resourceTableIDs.size() > 1) {
throw BinderException("Mixing relationships tables from an RDFGraph and regular relationship tables in one pattern is currently not supported.");
}
// ... rest of the code ...
}
Make sure to test the changes to confirm that the new error message is displayed when the exception is triggered.
src/binder/bind/bind_graph_pattern.cpp
This snippet contains the error message logic for relationship patterns, which is directly related to the issue of updating the error message when mixing RDFGraph and regular Property graph tables.
If I query this on a database with an RDFGraph and regular relationship tables I get the following error message:
Let's change the error message to:
... Mixing relationships tables from an RDFGraph and regular relationship tables in one pattern is currently not supported.
)