Skip to content

Commit 4552a80

Browse files
garyrussellartembilan
authored andcommitted
INT-4392: Fix tangles
JIRA: https://jira.spring.io/browse/INT-4392 JIRA: https://jira.spring.io/browse/INT-4391 JIRA: https://jira.spring.io/browse/INT-4393 Resolve class and leaf package tangles in: - core (except DSL) - file - http - move `StaticMessageHeaderAccessor` to root `integration` package - move acknowledgment related classes to a new `acks` package - move `MMIH` to `handler.support`, alongside argument resolvers, also `HandlerMethodArgumentResolversHolder` - move `ErrorMessagePublisher` to `core` - move `PatternMatchUtils` to `support.utils` - move `MessageSourceManagement` to `support.management` - remove direct reference to `MicrometerMetricsCapter` from `IntegrationManagementSupport` - add new class `MetricsCaptorLoader` - move `OperationsCallback` to inner interface in `RemoteFileOperations` - move `HttpContextUtils` to `config` * Simplify MetricsCaptor loading. * Polishing
1 parent 525eb00 commit 4552a80

File tree

52 files changed

+111
-115
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+111
-115
lines changed

spring-integration-amqp/src/main/java/org/springframework/integration/amqp/inbound/AmqpInboundChannelAdapter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,14 +29,14 @@
2929
import org.springframework.amqp.support.converter.SimpleMessageConverter;
3030
import org.springframework.core.AttributeAccessor;
3131
import org.springframework.integration.IntegrationMessageHeaderAccessor;
32+
import org.springframework.integration.StaticMessageHeaderAccessor;
3233
import org.springframework.integration.amqp.support.AmqpHeaderMapper;
3334
import org.springframework.integration.amqp.support.AmqpMessageHeaderErrorMessageStrategy;
3435
import org.springframework.integration.amqp.support.DefaultAmqpHeaderMapper;
3536
import org.springframework.integration.context.OrderlyShutdownCapable;
3637
import org.springframework.integration.endpoint.MessageProducerSupport;
3738
import org.springframework.integration.support.ErrorMessageStrategy;
3839
import org.springframework.integration.support.ErrorMessageUtils;
39-
import org.springframework.integration.support.StaticMessageHeaderAccessor;
4040
import org.springframework.retry.RecoveryCallback;
4141
import org.springframework.retry.RetryCallback;
4242
import org.springframework.retry.RetryContext;

spring-integration-amqp/src/main/java/org/springframework/integration/amqp/inbound/AmqpInboundGateway.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,13 @@
3434
import org.springframework.amqp.support.converter.SimpleMessageConverter;
3535
import org.springframework.core.AttributeAccessor;
3636
import org.springframework.integration.IntegrationMessageHeaderAccessor;
37+
import org.springframework.integration.StaticMessageHeaderAccessor;
3738
import org.springframework.integration.amqp.support.AmqpHeaderMapper;
3839
import org.springframework.integration.amqp.support.AmqpMessageHeaderErrorMessageStrategy;
3940
import org.springframework.integration.amqp.support.DefaultAmqpHeaderMapper;
4041
import org.springframework.integration.gateway.MessagingGatewaySupport;
4142
import org.springframework.integration.support.ErrorMessageStrategy;
4243
import org.springframework.integration.support.ErrorMessageUtils;
43-
import org.springframework.integration.support.StaticMessageHeaderAccessor;
4444
import org.springframework.retry.RecoveryCallback;
4545
import org.springframework.retry.RetryCallback;
4646
import org.springframework.retry.RetryContext;

spring-integration-amqp/src/main/java/org/springframework/integration/amqp/inbound/AmqpMessageSource.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,13 +31,13 @@
3131
import org.springframework.amqp.support.converter.SimpleMessageConverter;
3232
import org.springframework.context.MessageSource;
3333
import org.springframework.integration.IntegrationMessageHeaderAccessor;
34+
import org.springframework.integration.acks.AcknowledgmentCallback;
35+
import org.springframework.integration.acks.AcknowledgmentCallbackFactory;
3436
import org.springframework.integration.amqp.support.AmqpHeaderMapper;
3537
import org.springframework.integration.amqp.support.AmqpMessageHeaderErrorMessageStrategy;
3638
import org.springframework.integration.amqp.support.DefaultAmqpHeaderMapper;
3739
import org.springframework.integration.endpoint.AbstractMessageSource;
3840
import org.springframework.integration.support.AbstractIntegrationMessageBuilder;
39-
import org.springframework.integration.support.AcknowledgmentCallback;
40-
import org.springframework.integration.support.AcknowledgmentCallbackFactory;
4141
import org.springframework.util.Assert;
4242

4343
import com.rabbitmq.client.Channel;

spring-integration-amqp/src/test/java/org/springframework/integration/amqp/inbound/AmqpMessageSourceIntegrationTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,8 @@
4949
import org.springframework.context.annotation.Bean;
5050
import org.springframework.context.annotation.Configuration;
5151
import org.springframework.integration.IntegrationMessageHeaderAccessor;
52+
import org.springframework.integration.acks.AcknowledgmentCallback;
53+
import org.springframework.integration.acks.AcknowledgmentCallback.Status;
5254
import org.springframework.integration.amqp.dsl.Amqp;
5355
import org.springframework.integration.annotation.InboundChannelAdapter;
5456
import org.springframework.integration.annotation.Poller;
@@ -58,8 +60,6 @@
5860
import org.springframework.integration.dsl.IntegrationFlow;
5961
import org.springframework.integration.dsl.IntegrationFlows;
6062
import org.springframework.integration.dsl.Pollers;
61-
import org.springframework.integration.support.AcknowledgmentCallback;
62-
import org.springframework.integration.support.AcknowledgmentCallback.Status;
6363
import org.springframework.messaging.handler.annotation.Header;
6464
import org.springframework.messaging.support.MessageBuilder;
6565
import org.springframework.test.annotation.DirtiesContext;

spring-integration-amqp/src/test/java/org/springframework/integration/amqp/inbound/AmqpMessageSourceTests.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@
3434

3535
import org.springframework.amqp.rabbit.connection.CachingConnectionFactory;
3636
import org.springframework.amqp.support.AmqpHeaders;
37+
import org.springframework.integration.StaticMessageHeaderAccessor;
38+
import org.springframework.integration.acks.AcknowledgmentCallback.Status;
3739
import org.springframework.integration.amqp.support.AmqpMessageHeaderErrorMessageStrategy;
38-
import org.springframework.integration.support.AcknowledgmentCallback.Status;
39-
import org.springframework.integration.support.StaticMessageHeaderAccessor;
4040
import org.springframework.messaging.Message;
4141

4242
import com.rabbitmq.client.AMQP.BasicProperties;

spring-integration-amqp/src/test/java/org/springframework/integration/amqp/inbound/InboundEndpointTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@
5858
import org.springframework.amqp.support.converter.MessageConverter;
5959
import org.springframework.amqp.support.converter.SimpleMessageConverter;
6060
import org.springframework.beans.factory.BeanFactory;
61+
import org.springframework.integration.StaticMessageHeaderAccessor;
6162
import org.springframework.integration.amqp.support.AmqpMessageHeaderErrorMessageStrategy;
6263
import org.springframework.integration.amqp.support.DefaultAmqpHeaderMapper;
6364
import org.springframework.integration.channel.DirectChannel;
@@ -67,7 +68,6 @@
6768
import org.springframework.integration.json.ObjectToJsonTransformer;
6869
import org.springframework.integration.mapping.support.JsonHeaders;
6970
import org.springframework.integration.support.MessageBuilder;
70-
import org.springframework.integration.support.StaticMessageHeaderAccessor;
7171
import org.springframework.integration.transformer.MessageTransformingHandler;
7272
import org.springframework.integration.transformer.Transformer;
7373
import org.springframework.messaging.Message;

spring-integration-core/src/main/java/org/springframework/integration/IntegrationMessageHeaderAccessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
import java.util.Set;
2525
import java.util.concurrent.atomic.AtomicInteger;
2626

27-
import org.springframework.integration.support.AcknowledgmentCallback;
27+
import org.springframework.integration.acks.AcknowledgmentCallback;
2828
import org.springframework.lang.Nullable;
2929
import org.springframework.messaging.Message;
3030
import org.springframework.messaging.MessageHeaders;
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,13 @@
1414
* limitations under the License.
1515
*/
1616

17-
package org.springframework.integration.support;
17+
package org.springframework.integration;
1818

1919
import java.io.Closeable;
2020
import java.util.UUID;
2121
import java.util.concurrent.atomic.AtomicInteger;
2222

23-
import org.springframework.integration.IntegrationMessageHeaderAccessor;
23+
import org.springframework.integration.acks.AcknowledgmentCallback;
2424
import org.springframework.lang.Nullable;
2525
import org.springframework.messaging.Message;
2626
import org.springframework.messaging.MessageHeaders;
Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@
1414
* limitations under the License.
1515
*/
1616

17-
package org.springframework.integration.support;
17+
package org.springframework.integration.acks;
1818

19-
import org.springframework.integration.support.AcknowledgmentCallback.Status;
19+
import org.springframework.integration.acks.AcknowledgmentCallback.Status;
2020
import org.springframework.lang.Nullable;
2121
import org.springframework.messaging.Message;
2222

@@ -37,10 +37,12 @@ private AckUtils() {
3737
* Return the {@link AcknowledgmentCallback} header (if present).
3838
* @param message the message.
3939
* @return the callback, or null.
40+
* @deprecated use StaticMessageHeaderAccessor.getAcknowledgmentCallback(message).
4041
*/
42+
@Deprecated
4143
@Nullable
4244
public static AcknowledgmentCallback getAckCallback(Message<?> message) {
43-
return StaticMessageHeaderAccessor.getAcknowledgmentCallback(message);
45+
throw new UnsupportedOperationException("Use StaticMessageHeaderAccessor.getAcknowledgmentCallback(message)");
4446
}
4547

4648
/**
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
* limitations under the License.
1515
*/
1616

17-
package org.springframework.integration.support;
17+
package org.springframework.integration.acks;
1818

1919
/**
2020
* A general abstraction over acknowledgments.

0 commit comments

Comments
 (0)