-
Notifications
You must be signed in to change notification settings - Fork 65
Comparing changes
Open a pull request
base repository: maidsafe/qp2p
base: v0.22.1
head repository: maidsafe/qp2p
compare: v0.22.2
- 8 commits
- 12 files changed
- 2 contributors
Commits on Oct 13, 2021
-
ci: update
maidsafe/rust-version-bump-branch-creator
The new version maintains the format of `Cargo.toml`, so we should be able to tidy that up a bit.
Configuration menu - View commit details
-
Copy full SHA for 0d8455a - Browse repository at this point
Copy the full SHA 0d8455aView commit details -
chore: format
Cargo.toml
and simplifytokio
featuresHaving bumped `maidsafe/rust-version-bump-branch-creator`, the `Cargo.toml` format will be preserved, so we can tidy up the format. Additionally, only one of the `tokio` features were being used, apart from `macros` which was only needed for tests.
Configuration menu - View commit details
-
Copy full SHA for d79a01a - Browse repository at this point
Copy the full SHA d79a01aView commit details -
feat: add a counter for opened connections to
Endpoint
This is behind an `unstable-opened-connection-count` feature flag, so that it can be removed again in future without a breaking change. The intention is to use it to get a measure of the current number of connections that we open in downstream tests, in order to see the effect of removing the `ConnectionPool` and track down extraneous connections.
Configuration menu - View commit details
-
Copy full SHA for c6a9a42 - Browse repository at this point
Copy the full SHA c6a9a42View commit details -
refactor: centralise definition of certificate server name
This was declared both in `config` and `endpoint`, but is now exported from `config` and imported in `endpoint`.
Configuration menu - View commit details
-
Copy full SHA for c88fbdb - Browse repository at this point
Copy the full SHA c88fbdbView commit details -
refactor: store less config in
Endpoint
Most of the values in `InternalConfig` are only used when constructing the endpoint. In fact, it turns out that only the `RetryConfig` needs to be remembered (since it gets passed to all opened `Connection`s).
Configuration menu - View commit details
-
Copy full SHA for 6b79435 - Browse repository at this point
Copy the full SHA 6b79435View commit details -
refactor: rename
connections
->connection_handle
This is an internal refactor only, in preparation for replacing most of the connection machinery with an unpooled implementation.
Configuration menu - View commit details
-
Copy full SHA for cc6e9fc - Browse repository at this point
Copy the full SHA cc6e9fcView commit details -
refactor: implement a non-pooled
Connection
This commit adds a `connection` module, which implements a connection API with no dependencies on the connection pool. Most of the API has moved from `connection_handle`, but from now `ConnectionHandle` is still exported as `Connection`. When used on its own, the new `Connection` API follows the typical drop-based lifecycle. Once both sides of the API (`Connection` and `ConnectionIncoming`) are dropped, the connection will close automatically. However, for compability in the short term the `Endpoint` APIs will still put these `Connection`s in the connection pool, and poll for incoming messages in a background task, so for now connections will still remain open until error or explicit close.
Configuration menu - View commit details
-
Copy full SHA for 8eaea98 - Browse repository at this point
Copy the full SHA 8eaea98View commit details -
Configuration menu - View commit details
-
Copy full SHA for a6081f6 - Browse repository at this point
Copy the full SHA a6081f6View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff v0.22.1...v0.22.2