NSAmelchev opened a new pull request, #13788:
URL: https://github.com/apache/ignite/pull/13788
### Changes
**`GridDhtGetFuture`, `GridDhtGetSingleFuture`**
Removed the unused `futId` and `ver` fields together with their getters.
Nothing reads them: both classes are `final`, callers hold the futures as
`GridDhtFuture` / `IgniteInternalFuture`.
**`GridClusterStateProcessor#publicApiState`**
Returns the current state directly when the cluster is not in transition or
the target state is inactive — exactly the cases where `publicApiStateAsync`
returns a finished future. `publicApiActiveState` now delegates to it, so the
condition lives in one place.
### Visible behaviour change
The version counter is no longer advanced by reads, so the `LastDataVersion`
metric changes only on writes, which matches its description ("The latest data
version on the node").
### Measurement
`JmhCacheRemoteGetBenchmark`: single-key `get`, 4 threads, `ATOMIC`,
`backups=0`, 2 servers + 1 client in one JVM. `gc.alloc.rate.norm`, median of 3
JVMs per variant:
| Benchmark | master | patch | Δ |
|---|---|---|---|
| client get | 2677 B/op | 2612 B/op | −2.4 % |
| server get, remote key | 2674 B/op | 2497 B/op | −6.6 % |
| server get, local key | 992 B/op | 920 B/op | −7.3 % |
Throughput is within run-to-run noise.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
