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

Do not traverse element hierarchy during processChanges #98

Open
MichaelBelousov opened this issue Jul 19, 2023 · 1 comment
Open

Do not traverse element hierarchy during processChanges #98

MichaelBelousov opened this issue Jul 19, 2023 · 1 comment
Labels
performance potential performance increase

Comments

@MichaelBelousov
Copy link
Contributor

The initial design of the transformer required traversing the iModel in element order to perform the processChanges function. This is no longer necessary now that elements are not deferred. There is a potential speed up in directly exporting elements that are only in the change data instead of the entire hierarchy.

@MichaelBelousov
Copy link
Contributor Author

also, it is possible to do a one pass transformation if we do it in order. We would need to turn off foreign key constraints, and then add the target's starting id to each id that we move. Added elements would need to be added after the data, and removed elements would leave gaps in the id space.

@MichaelBelousov MichaelBelousov added the performance potential performance increase label Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
performance potential performance increase
Projects
None yet
Development

No branches or pull requests

1 participant