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

Add tags to a note is extrem slow #4251

Closed
JackGruber opened this issue Dec 27, 2020 · 10 comments · Fixed by #10331
Closed

Add tags to a note is extrem slow #4251

JackGruber opened this issue Dec 27, 2020 · 10 comments · Fixed by #10331
Assignees
Labels
bug It's a bug desktop All desktop platforms medium Medium priority issues

Comments

@JackGruber
Copy link
Contributor

Adding Tags to notes is extrem slow (nearly one second for each Tag) if you have a lot of tags/notes.

Environment

Joplin 1.5.7 (prod, win32)

Client ID: fea0ba97613b45d8b1c8ee33e9f08413
Sync Version: 2
Profile Version: 34
Keychain Supported: Yes

Revision: f1bbcea (master)

Sync status (synced items / total items)

Note: 5318/5318
Folder: 48/48
Resource: 8945/8945
Tag: 1697/1697
NoteTag: 18333/18333
MasterKey: 0/0
Revision: 562/562
Total: 34903/34903

Steps to reproduce

  1. Open a note
  2. Add some Tas
  3. Save

0e54e8d8e2a16a24c6648afdb1ae130164afc4c5

All this in an dev installation with only a few notes and tags is much faster.
3dc4d73a0fa9eeed9b2a4e9cb88e34042a386d3e

Describe what you expected to happen

Adding tags should work significantly faster and not take almost 1 second per tag.

Logfile

console.txt
log.txt

@JackGruber JackGruber added the bug It's a bug label Dec 27, 2020
@laurent22
Copy link
Owner

Hmm, yes that's not great. Although I'd be tempted to put that aside for now, since if we add support for hierarchical tags we'd need to review all this anyway and work on the performances.

When it takes one second to display each tag, is it frozen in the meantime or can you do something else. What if you add the tags, close the note, and come back. Are the tags saved?

@JackGruber
Copy link
Contributor Author

JackGruber commented Jan 5, 2021

If you select another note, it jumps back after each tag was added.
Therefore, during this time Joplin is not usable.

@stale
Copy link

stale bot commented Feb 4, 2021

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@stale stale bot added the stale An issue that hasn't been active for a while... label Feb 4, 2021
@JackGruber
Copy link
Contributor Author

Not fixed

@stale stale bot removed the stale An issue that hasn't been active for a while... label Feb 4, 2021
@roman-r-m
Copy link
Collaborator

Another, probably related, issue:

  • Select a few notes (the more, the better) and add a few tags to them.
  • At this point Joplin becomes almost unresponsive

Additionally, trying to open Tools > Options produces this exception in console

main-html.js:48 Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.
    in NoteEditor (created by Connect(NoteEditor))
    in Connect(NoteEditor) (created by ResizableLayout)
    in div (created by styled.div)
    in styled.div (created by ResizableLayout)
    in div (created by ResizableLayout)
    in div (created by ResizableLayout)
    in ResizableLayout (created by MainScreenComponent)
    in div (created by MainScreenComponent)
    in MainScreenComponent (created by Connect(MainScreenComponent))
    in Connect(MainScreenComponent) (created by NavigatorComponent)

@stale
Copy link

stale bot commented Mar 20, 2021

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@stale stale bot added the stale An issue that hasn't been active for a while... label Mar 20, 2021
@JackGruber
Copy link
Contributor Author

Still a issue

@stale stale bot removed the stale An issue that hasn't been active for a while... label Mar 20, 2021
@roman-r-m roman-r-m added backlog We'll get to it... eventually... desktop All desktop platforms labels Mar 25, 2021
@Jip-Hop
Copy link

Jip-Hop commented Apr 1, 2021

I just selected 1767 notes and added one tag to all of them. I can see in the tags sidebar on the left that the tags are being applied to the notes one by one at a rate of 1 tag per second. Currently waiting for 13 minutes and it's not half way done.

Joplin for Desktop
Joplin 1.7.11 (prod, darwin)
Sync Version: 2
Profile Version: 34
Keychain Supported: Yes

Revision: f560563 (master)

If it matters, it's also syncing simultaneously with the Joplin server on Docker.

@personalizedrefrigerator
Copy link
Collaborator

Related to https://discourse.joplinapp.org/t/performance-issue/37207/23

@personalizedrefrigerator personalizedrefrigerator removed the backlog We'll get to it... eventually... label Apr 17, 2024
@personalizedrefrigerator
Copy link
Collaborator

This also seems to be making sync and JEX import much slower when importing/downloading many tags and notebooks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug desktop All desktop platforms medium Medium priority issues
Projects
None yet
5 participants