Build #2,669

Build: #2669 was successful Changes by Simon Basle

Build result summary

Details

Completed
Duration
7 minutes
Labels
None
Agent
bamboo-24.af.pivotal.io
Revision
cf8ce81a9491b09f38152257bfaebfd3328d97a5 cf8ce81a9491b09f38152257bfaebfd3328d97a5
Total tests
5711
Successful since
#2660 ()

Tests

Code commits

Author Commit Message Commit date
Simon Basle Simon Basle cf8ce81a9491b09f38152257bfaebfd3328d97a5 cf8ce81a9491b09f38152257bfaebfd3328d97a5 fix #1970 Avoid creating Map.Entry in Context.putAll
This commit introduces `CoreContext` as a common marker for all
reactor-provided `Context` implementations.

This allows to invert the control of the putAll method, avoiding
external iteration via the `stream()` and thus wasteful production of
`Map.Entry` objects.

Inner specialized methods allow to bypass stream-based iteration:
 - `Context putAllInto(Context)` for the smaller size cases
 - `void unsafePutAllInto(ContextN)` for the special N case
   where a new ContextN is created and mutated by the source

The indirection also allows to avoid `instanceof` in `isEmpty`.
New tests around these cases ensure full test coverage.

Reviewed-In: #1971