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.

Sign up for GitHub

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

Jump to bottom

Speed up content on import and report #171

Closed
KevinJump opened this issue Oct 6, 2020 · 1 comment · Fixed by #175
Closed

Speed up content on import and report #171

KevinJump opened this issue Oct 6, 2020 · 1 comment · Fixed by #175
Labels
release/8.7.2

Comments

@KevinJump
Copy link
Owner

Content reporting has slowed down a little in the last few releases - mainly because of a lot of extra features we've added like missing parent checks #90 and variant / segment #148 updates.

however the report/import process could probably be improved a little, for example a report will serialize the same item 3 times, (one for comparison, and two to build the list of tracked changes!). we can fix this so it only does it once.

We are also considering adding the option of removing the missing parent check on a report - it would still work on imports but you wouldn't get missing parent in the report (missing parent does a full tree track tring to find a piece of content by a path of aliases, so the deeper your tree goes the slower the lookup.
KevinJump pushed a commit that referenced this issue Oct 6, 2020
ISyncNodeSerializer, ISyncNodeTracker - speeding up import #171
3999761
@KevinJump
Copy link
Owner Author

#171 reduces reserialization and significantly improves content imports when there are missing elements.

@KevinJump KevinJump linked a pull request Oct 13, 2020 that will close this issue
V8/less serialization #175
Merged
KevinJump added a commit that referenced this issue Oct 21, 2020
@KevinJump
V8/less serialization (#175)
ed2482b 
* Stop extra lookups of Root when content is missing.

* ISyncNodeSerializer, ISyncNodeTracker - speeding up import #171

* perf: improved parent path lookup

* Fix #142 - HashCode isn't deterministic

* Improve the tracker - so things that are matched on key then alias, don't show up as delete/create

* Update contentType tracker - track strutures by alias

* Change seperator for property path in tracker

* code cleanup (usings)

* add clear history button, make history load on tab view.

* Sync history events, to reload after import/export.

* Add key only serialization option #176

Adds a KeysOnly setting for the dictionary serializer

* Support for media in the grid styles.

Supports media inside the grid styles elements.

* Prerelease fix - path gets muddled when its in the cache

* prerelease fix: don't include root in friendly path.

* version bump. v8.7.2

* move version back to description

* Add option to ignore config by editorAlias or item name #178

* Generic IgnoreAlias setting for all handlers - stops them being imported completely.  #178

* up the version cache numbers for the js.

* Don't show addons if uSync.Complete is installed.

* Allow uSync folder to be set outside the site root #173

* Prerelease fix - Blueprint path fix.

* 8.7.2 version/readme.

Co-authored-by: Kevin Jump <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
release/8.7.2
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

V8/less serialization KevinJump/uSync
1 participant
@KevinJump