Skip to content

Database cleanup#39

Merged
dmgcodevil merged 7 commits intomainfrom
database-cleanup
Apr 5, 2026
Merged

Database cleanup#39
dmgcodevil merged 7 commits intomainfrom
database-cleanup

Conversation

@dmgcodevil
Copy link
Copy Markdown
Owner

No description provided.

* @param edge_type The edge type name (e.g. "WORKS_AT").
* @return The shadow schema name, or an error if the edge schema is missing.
*/
arrow::Result<std::string> ensure_edge_shadow_schema(
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • [x]

apply_updates(alias_to_schema.at(alias), id_column, fields,
uq.update_type(), result);
} else {
auto src_col = table->GetColumnByName(trav->source().value() + ".id");
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to remove hardcoded "id" field. task

@dmgcodevil dmgcodevil merged commit c86fb38 into main Apr 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant