Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor changes to "property" table #215

Closed
clausnagel opened this issue Nov 24, 2023 · 2 comments
Closed

Minor changes to "property" table #215

clausnagel opened this issue Nov 24, 2023 · 2 comments
Assignees
Labels

Comments

@clausnagel
Copy link
Member

clausnagel commented Nov 24, 2023

  • Drop root_id column
    This column makes it easy to query all rows belonging to the same property hierarchy. The same can be achieved using an hierarchical query on parent_id though. Dropping this column would also remove the corresponding FK constraint and index both of which are expensive during imports.

  • Drop FK constraint on "datatype_id"
    This constraint is expensive during imports. Removing it would allow storing values not defined in the "datatype" table. However, such invalid values can simply be ignored in exports and queries.

  • Drop FK constraint on "namespace_id"
    This constraint is expensive during imports. Removing it would allow storing values not defined in the "namespace" table. However, such invalid values can simply be ignored in exports and queries. The index on "namespace_id" must be kept though.

@clausnagel clausnagel added the v5 label Nov 24, 2023
@clausnagel
Copy link
Member Author

  • To be consistent, also drop the FK constraint on the "objectclass_id" column of the "feature" table. The index on "objectclass_id" must be kept though.

@yaozhihang
Copy link
Member

done with a1384f4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants