Gary Russell <grussell@vmware.com>: Author Summary

Builds triggered by Gary Russell <grussell@vmware.com>

Builds triggered by an author are those builds which contains changes committed by the author.
374
49 (13%)
325 (87%)

Breakages and fixes

Broken means the build has failed but the previous build was successful.
Fixed means that the build was successful but the previous build has failed.
34 (9% of all builds triggered)
26 (7% of all builds triggered)
-8
Build Completed Code commits Tests
SK › K26X › #42 9 hours ago
GH-1776: Fix transactional Template with Overrides
Resolves https://github.com/spring-projects/spring-kafka/issues/1776

Set `transactional` flag using overrides.

**cherry-pick to 2.6.x, 2.5.x**
443 passed
SK › K25X › #101 9 hours ago
GH-1776: Fix transactional Template with Overrides
Resolves https://github.com/spring-projects/spring-kafka/issues/1776

Set `transactional` flag using overrides.

**cherry-pick to 2.6.x, 2.5.x**
421 passed
SK › MAS › #1383 9 hours ago
GH-1776: Fix transactional Template with Overrides
Resolves https://github.com/spring-projects/spring-kafka/issues/1776

Set `transactional` flag using overrides.

**cherry-pick to 2.6.x, 2.5.x**
575 passed
SK › MAS › #1382 13 hours ago
Javadoc Polishing
575 passed
SK › MAS › #1381 13 hours ago
GH-1774: kafka-clients 2.8.0 Compatibility
Resolves https://github.com/spring-projects/spring-kafka/issues/1774

- changes to `BrokerState`
- suppress some deprecation warnings

* Remove unnecessary static field.

* Remove unnecessary warning suppressions.
575 passed
INT › MAINNIGHTLY › #9 5 days ago
(S)FTP Doc Polishing
5332 passed
INT › SONAR › #3568 5 days ago
(S)FTP Doc Polishing
Testless build
SK › K26X › #41 5 days ago
GH-1770: Fix ClientId for @KafkaListener
Resolves https://github.com/spring-projects/spring-kafka/issues/1770

The clientId property, if set on the container factory, is overwritten
by an empty String if not defined on the annotation.

Check for an empty String (groupId too).

**cherry-pick to 2.6.x**

* Use simpler JavaUtils method.
443 passed
SK › MAS › #1380 5 days ago
GH-1770: Fix ClientId for @KafkaListener
Resolves https://github.com/spring-projects/spring-kafka/issues/1770

The clientId property, if set on the container factory, is overwritten
by an empty String if not defined on the annotation.

Check for an empty String (groupId too).

**cherry-pick to 2.6.x**

* Use simpler JavaUtils method.
575 passed
INT › MAIN › #14 5 days ago
(S)FTP Doc Polishing
2 of 4547 failed
Build Completed Code commits Tests
INT › SONAR › #3568 5 days ago
(S)FTP Doc Polishing
Testless build
INT › MAIN › #14 5 days ago
(S)FTP Doc Polishing
2 of 4547 failed
INT › SONAR › #3564 1 week ago
GH-3520: Improve TcpConnectionInterceptor Javadocs
Resolves https://github.com/spring-projects/spring-integration/issues/3520
Testless build
AMQP › R22X › #84 2 weeks ago
GH-1315: Add onFailed() to ConnectionListener
Resolves https://github.com/spring-projects/spring-amqp/issues/1315

**cherry-pick to 2.2.x**

* Fix `@since` in the new test class
159 passed
INT › SI54X › #39 3 weeks ago
GH-3526: Fix Infinite Loop in FailoverCConnFactory
Resolves https://github.com/spring-projects/spring-integration/issues/3526

`FailoverClientConnectionFactory`

The logic to detect we had iterated over all factories and including
the one from which the previous connection was established was incorrect,
causing an infite loop until one of the factory connections was successful.

Change the logic to detect we have reset the iterator and the current failure
is from the same factory as the one from which the previous connection was
established.

**cherry-pick to 5.4.x, 5.3.x**

* Add diagnostics.

* Fix race in test.

* More race fixes and diagnostics.

* Remove diagnostics.
3306 passed
INT › SI53X › #86 3 weeks ago
GH-3526: Fix Infinite Loop in FailoverCConnFactory
Resolves https://github.com/spring-projects/spring-integration/issues/3526

`FailoverClientConnectionFactory`

The logic to detect we had iterated over all factories and including
the one from which the previous connection was established was incorrect,
causing an infite loop until one of the factory connections was successful.

Change the logic to detect we have reset the iterator and the current failure
is from the same factory as the one from which the previous connection was
established.

**cherry-pick to 5.4.x, 5.3.x**

* Add diagnostics.

* Fix race in test.

* More race fixes and diagnostics.

* Remove diagnostics.
3083 passed
INT › SONAR › #3543 4 weeks ago
GH-3523: TcpConnectionEvent Fixes
Resolves https://github.com/spring-projects/spring-integration/issues/3523

- `TcpNetConnection` - publish open event on reader thread to avoid race with first read.
- Intercepted connections - Ensure that the event source is always the outermost interceptor.
- Also reduce delays during NIO client connect.

**cherry-pick to 5.4.x, 5.3.x**

5.3.x will require cherry picking GH-3509 commits.
Testless build
SK › MAS › #1349 4 weeks ago
Remove Invalid Reactive Test
- author misunderstood `sendOffsetsToTransaction`; the test sends the offsets
  for a produced record.
15 passed
SK › MAS › #1346 4 weeks ago
Don't Log Converted Message With Projection
`toString()` cannot be invoked on the payload proxy created by
the project factory.
1 of 568 failed
INT › MASTER › #2365 1 month ago
GH-3509: Fix Memory Leak with Intercepted TCP Conn
Resolves https://github.com/spring-projects/spring-integration/issues/3509

When a connection was intercepted, the `TcpSender.add...` was called with the
interceptor, but the `removeDead...` was called with the actual connection,
causing a memory leak.

Always call the `TcpSender` with the outer-most interceptor.

# Conflicts:
#        spring-integration-ip/src/main/java/org/springframework/integration/ip/tcp/connection/TcpNetServerConnectionFactory.java
#        spring-integration-ip/src/test/java/org/springframework/integration/ip/tcp/TcpSendingMessageHandlerTests.java
4 of 3460 failed
Build Completed Code commits Tests
INT › MAINNIGHTLY › #6 1 week ago
GH-3520: Improve TcpConnectionInterceptor Javadocs
Resolves https://github.com/spring-projects/spring-integration/issues/3520
5332 passed
INT › SONAR › #3545 3 weeks ago
GH-3526: Fix Infinite Loop in FailoverCConnFactory
Resolves https://github.com/spring-projects/spring-integration/issues/3526

`FailoverClientConnectionFactory`

The logic to detect we had iterated over all factories and including
the one from which the previous connection was established was incorrect,
causing an infite loop until one of the factory connections was successful.

Change the logic to detect we have reset the iterator and the current failure
is from the same factory as the one from which the previous connection was
established.

**cherry-pick to 5.4.x, 5.3.x**

* Add diagnostics.

* Fix race in test.

* More race fixes and diagnostics.

* Remove diagnostics.
Testless build
INT › SI54X › #40 3 weeks ago
GH-536: Fix test for backport
5165 passed
INT › SI53X › #87 3 weeks ago
GH-536: Fix test for backport
5048 passed
SK › MAS › #1350 4 weeks ago
Checkstyle Fixes
565 passed
SK › MAS › #1347 4 weeks ago
Diagnostics for Flaky Reactor Test
566 passed
INT › MASTERSPRING40 › #1386 1 month ago
GH-3509: Fix if test in previous commit
Caused interceptor to be skipped.

**cherry-pick to 5.4.x**

* Fix race in new tests

* Remove overloaded conn id in test interceptor.
GH-3509: Fix Memory Leak with Intercepted TCP Conn
Resolves https://github.com/spring-projects/spring-integration/issues/3509

When a connection was intercepted, the `TcpSender.add...` was called with the
interceptor, but the `removeDead...` was called with the actual connection,
causing a memory leak.

Always call the `TcpSender` with the outer-most interceptor.

# Conflicts:
#        spring-integration-ip/src/main/java/org/springframework/integration/ip/tcp/connection/TcpNetServerConnectionFactory.java
#        spring-integration-ip/src/test/java/org/springframework/integration/ip/tcp/TcpSendingMessageHandlerTests.java
5326 passed
INT › SI54X › #34 1 month ago
GH-3509: Fix if test in previous commit
Caused interceptor to be skipped.

**cherry-pick to 5.4.x**

* Fix race in new tests

* Remove overloaded conn id in test interceptor.
5161 passed
INT › MASTER › #2366 1 month ago
GH-3509: Fix if test in previous commit
Caused interceptor to be skipped.

**cherry-pick to 5.4.x**

* Fix race in new tests

* Remove overloaded conn id in test interceptor.
5190 passed
SK › MAS › #1337 1 month ago
GH-1736: Depr. StreamsBuilderFactoryBeanCustomizer
Resolves https://github.com/spring-projects/spring-kafka/issues/1736

**cherry-pick to 2.6.x**
559 passed