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

remove network migration and optimize network event retry #3510

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

woutslakhorst
Copy link
Member

@woutslakhorst woutslakhorst commented Oct 22, 2024

Removed all logic referred to by remove for v5/v6 release

Optimised network event retry at startup causing deadlock between long running read and write lock

@woutslakhorst woutslakhorst force-pushed the fix/network_event_retry_locking branch from d17ecee to 9db4bc3 Compare October 22, 2024 12:36
}

// Migrate headsLegacy to single head
// Todo: remove after V6 release => then remove headsShelf
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm assuming the second half of this todo points to a now unused constant that should be removed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused shelf, but since the entire DB will be removed in the future...

@woutslakhorst woutslakhorst merged commit 08ad0e5 into master Oct 22, 2024
9 checks passed
@woutslakhorst woutslakhorst deleted the fix/network_event_retry_locking branch October 22, 2024 13:09
rolandgroen added a commit that referenced this pull request Nov 1, 2024
* master: (72 commits)
  PKI add ValidateStrict (#3531)
  PEX: Return capture group for matched patterns (#3526)
  Schedule CodeQL twice a week (#3525)
  change cron schedule (#3524)
  Add gh action for CodeQL schedule (#3523)
  Bump github.com/lestrrat-go/jwx/v2 from 2.1.1 to 2.1.2 (#3520)
  docs: v6 release date (#3519)
  status codes for discovery client (#3513)
  Require SQL connection string in strictmode (#3517)
  Fix duplicate discovery results (#3515)
  Bump github.com/chromedp/chromedp from 0.11.0 to 0.11.1 (#3514)
  fix duplicate search results for wildcard param (#3512)
  Bump github.com/Azure/azure-sdk-for-go/sdk/security/keyvault/azkeys (#3511)
  remove network migration and optimize network event retry (#3510)
  secure outgoing http client with max connections (#3508)
  make gen-mocks (#3509)
  Bump github.com/Azure/azure-sdk-for-go/sdk/azcore from 1.15.0 to 1.16.0 (#3506)
  fix invalid keyReference migration objects (#3504)
  Bump go.uber.org/mock from 0.4.0 to 0.5.0 (#3507)
  Bump github.com/nats-io/nats-server/v2 from 2.10.21 to 2.10.22 (#3505)
  ...

# Conflicts:
#	vcr/test.go
#	vdr/legacy_integration_test.go
#	vdr/vdr.go
#	vdr/vdr_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants