Build: #3 failed Scheduled with changes by Alexander Pinske <[email protected]> and Artem Bilan

Code commits

Plan Repository

  • Artem Bilan

    Artem Bilan c811da6dd8c5a3ea3575f6a79332d54658c5d9ae

    Fix compatibility with Java 8 for ByteBuffer (#3545)
    When we build with Java 9+ and target for Java 8,
    we get an incompatible bytecode around these method for `ByteBuffer`:

    ```
    position(int)
    limit(int)
    mark()
    reset()
    clear()
    flip()
    rewind()
    ```

    The recommendation is to cast to `Buffer` when we call these methods

    * Fix all the production code using `ByteBuffer` for recommended cast to `Buffer`

    Related to https://build.spring.io/browse/INTEXT-AWS-306

    See more info in this Jetty issue: https://github.com/eclipse/jetty.project/issues/3244

    • spring-integration-core/src/main/java/org/springframework/integration/support/json/EmbeddedJsonHeadersMessageMapper.java (version c811da6dd8c5a3ea3575f6a79332d54658c5d9ae)
    • spring-integration-ip/src/main/java/org/springframework/integration/ip/tcp/connection/TcpNioSSLConnection.java (version c811da6dd8c5a3ea3575f6a79332d54658c5d9ae)
    • spring-integration-ip/src/main/java/org/springframework/integration/ip/udp/DatagramPacketMessageMapper.java (version c811da6dd8c5a3ea3575f6a79332d54658c5d9ae)
  • Alexander Pinske <alex@pinske.eu>

    Alexander Pinske <[email protected]> 3c27241d31a08e84753732fb4aaa900e7e328f66

    Improve IMAP IDLE tests
    This forces the race condition that was fixed in #3535 consistently.
    Before the `IdleCanceler` was used to cancel the `idle()` call in addition to the normal flow.
    That should not be necessary any more.

    • spring-integration-mail/src/test/java/org/springframework/integration/mail/ImapMailReceiverTests.java (version 3c27241d31a08e84753732fb4aaa900e7e328f66)
    • spring-integration-test-support/src/main/java/org/springframework/integration/test/mail/TestMailServer.java (version 3c27241d31a08e84753732fb4aaa900e7e328f66)
  • Artem Bilan

    Artem Bilan d8c7ae7b2f699ece2fd7019dedc96cb8945c22d6

    Revise retry for `IntegrationReactiveUtils` (#3543)
    Related to: https://github.com/spring-cloud/stream-applications/issues/156

    The `IntegrationReactiveUtils` uses a general `Flux.retry()`
    operator to always retry for all the errors.
    On the other hand it has only a `.doOnError(MessagingException.class)`
    which leads swallowing all the other exceptions from logs and handling.

    * Replace with `retryWhen()` for the `MessagingException` predicate
    failing for all other exceptions.
    The end-user may add their own `retry` or error handling mechanism to
    the returned `Flux` from the `IntegrationReactiveUtils`

    **Cherry-pick to 5.4.x**

    • spring-integration-core/src/main/java/org/springframework/integration/util/IntegrationReactiveUtils.java (version d8c7ae7b2f699ece2fd7019dedc96cb8945c22d6)
    • spring-integration-core/src/test/java/org/springframework/integration/channel/IntegrationReactiveUtilsTests.java (version d8c7ae7b2f699ece2fd7019dedc96cb8945c22d6)
  • Artem Bilan

    Artem Bilan aa32270c5b35fa765870ba4cd9ed24da228e71ba

    GH-3541: Properly guard when no json-path in CP (#3542)
    Fixes https://github.com/spring-projects/spring-integration/issues/3541

    * Fix `DefaultConfiguringBeanFactoryPostProcessor` to check for `jsonPath` bean
    a `com.jayway.jsonpath.JsonPath` on CP instead of `JsonPathUtils`
    which is always there since it is a part of `spring-integration-core`
    * Remove `json-path` and `jackson-databind` from tests classpath
    to be sure that we have coverage for the mentioned above option classpath entries
    * Add `json-path` and `jackson-databind` into test classpath whenever it is necessary

    • build.gradle (version aa32270c5b35fa765870ba4cd9ed24da228e71ba)
    • spring-integration-core/src/main/java/org/springframework/integration/config/DefaultConfiguringBeanFactoryPostProcessor.java (version aa32270c5b35fa765870ba4cd9ed24da228e71ba)
    • spring-integration-jmx/src/test/java/org/springframework/integration/jmx/configuration/EnableMBeanExportTests.java (version aa32270c5b35fa765870ba4cd9ed24da228e71ba)