-
Notifications
You must be signed in to change notification settings - Fork 333
Conversation
Thanks! I'm excited to merge this, do you mind if I leave it open for a couple days just to make sure we fix #1253 before adding indexing to the mix? |
No please. No rush |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried this now and stuff works for me. I don't know exactly what the index is used for though so I might miss something. And gonna let Christian click merge on this (idk the release schedule or whatever you wanna call it :p ).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
Got infinite scuttling. Idk if it's related to this PR or conn-stuff but anyway. A lot of logging, I'll post what I can.
and also a lot of
|
cc @staltz idk if you recognize those stack traces as being conn-related (I mean it doesn't mention conn but maybe an error that's propagated?) |
@Powersource doesn't seem ssb-conn related to me. Those kind of errors have always been somewhat common in SSB since always. |
First of all, thanks for testing @Powersource I'm with Staltz on the errors. These look like network errors? What happens if you run a
|
Unfortunately started the regular client instead of this branch again so didn't get to see the index status. Will try to run this branch again now and try to reproduce. Any way I agree that this issue is probably unrelated to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since i don't think there's a button to remove my "request changes"
I don't know whether I remember 100% -- when we bump indexes should we be doing releases of all clients? I'm worried that if Patchwork releases with a new SSB-Backlinks then if someone opens and closes Patchwork and then switches to another client then they'll just keep deleting the backlinks directory and starting from scratch. Is that true, or am I tripping? |
Yes, patchbay is already upgraded ;) |
Sweet, I'm down to merge ASAP. It looks like |
Thanks @christianbundy. I pushed a merge now. |
I don't think it's a merge conflict, it looks like a conflict between |
Thanks. Latest commit should do it. |
🎉 |
This should be much faster especially for onboarding people. See this issue for details.