malhuda/alice-lg
1
0
Fork 0
Code Issues 39 Pull Requests 11 Actions Packages Projects Releases 2 Wiki Activity

Concurrent update of Neighbours and Routes #49

Closed
zromi18 wants to merge 1 commits from develop into develop
pull from: develop
merge into: malhuda:develop
Conversation 5 Commits 1 Files Changed 5 +186 -110
zromi18 commented 2020-12-31 16:16:34 +08:00 (Migrated from github.com)
Copy Link

In the case of big infrastructures with many route servers monitored with Bird Watcher, the sequential update of Neighbours and Routes takes lots of time. Therefore to improve the performance it is better to do this in a concurrent manner

In the case of big infrastructures with many route servers monitored with Bird Watcher, the sequential update of Neighbours and Routes takes lots of time. Therefore to improve the performance it is better to do this in a concurrent manner
annikahannig commented 2021-01-01 01:21:34 +08:00 (Migrated from github.com)
Copy Link

Hey! Thanks for the patch! :)

Hey! Thanks for the patch! :)
annikahannig commented 2021-01-26 01:38:49 +08:00 (Migrated from github.com)
Copy Link

(not forgotten - I'll have a look at it in the next couple of days :) )

(not forgotten - I'll have a look at it in the next couple of days :) )
daswafford commented 2021-04-22 05:30:55 +08:00 (Migrated from github.com)
Copy Link

This is a good change. I also need to make the same to improve performance for my own Alice installation. @annikahannig could you take a fresh look at this one? Thank you.

This is a good change. I also need to make the same to improve performance for my own Alice installation. @annikahannig could you take a fresh look at this one? Thank you.
annikahannig commented 2021-04-26 17:44:03 +08:00 (Migrated from github.com)
Copy Link

I have a bit of a high workload right now - have you tested this?

I have a bit of a high workload right now - have you tested this?
annikahannig commented 2021-04-26 17:45:04 +08:00 (Migrated from github.com)
Copy Link

(I tend to be a bit cautious when it comes to things concurrency related - as there have been issues with syncing in the past)

(I tend to be a bit cautious when it comes to things concurrency related - as there have been issues with syncing in the past)

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something isn't working
dependencies
Pull requests that update a dependency file
duplicate
This issue or pull request already exists
enhancement
New feature or request
go
Pull requests that update Go code
good first issue
Good for newcomers
help wanted
Extra attention is needed
invalid
This doesn't seem right
javascript
Pull requests that update Javascript code
priority
question
Further information is requested
wontfix
This will not be worked on
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: malhuda/alice-lg#49
No description provided.
Delete Branch "develop"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?