Commit Graph

3981 Commits

Author SHA1 Message Date
Oleg Zhurakousky
116813d1bb Add test for empty JSON 2023-11-30 19:01:00 +01:00
buildmaster
b5476231ce Bumping versions 2023-11-29 10:15:15 +00:00
Chris Bono
fe6d8871f4 Fix failure in PulsarExtendedBindingPropertiesTests
The default value for PulsarContainerProperties.subscriptionType was switched
to `null` in ed3899fcb9 (diff-1371e1026362d375ef0f90846291add7cdcae0c340b934e3972d0bd72b312fa5L61).

This commit adjusts the PulsarExtendedBindingPropertiesTests
'extendedBindingsArePropagatedToContainerProperties' test for this fact.
2023-11-27 10:29:12 -05:00
Soby Chacko
d30f94af46 GH-2857: Kafka Streams binder customization issues
- StreamsBuilderFactoryBeanConfigurer customizaton happpens only after
   the StreamsBuilder object is created by the factory bean. This creates
   a scenario in which the customizations provided by the application are
   skipped by the StreamsBuilder. Addressing this issue.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2857
2023-11-22 18:41:34 -05:00
Soby Chacko
652b22cf76 Fix failing tests
- Fix failing Kafka Streams binder tests due to some EmbeddedKafka changes
2023-11-22 18:21:21 -05:00
Soby Chacko
169d56be2c GH-2849: Pollable consumers bindings endpoint
- With pollable consumers, bindings actuator endpoint throws
   an exception since it cannot find the binding key in the bindings
   map (the key is stroed with the actual destination topic).
   Addressing this issue by quering the binding with the binding name.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2849
2023-11-20 13:33:33 +01:00
Soby Chacko
06af5ceeb0 GH-2848: Partitioning issues in StreamBridge
- When output-bindings property is explicitly provided and native-encoding
   is used, StreamBridge does not add the partitioning interceptor.
   This commit addresses this issue.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2848
2023-11-20 13:32:59 +01:00
Georg Friedrich
ba0924acc2 GH-2843: Kafka Streams binder component bean issue
Fix automatic Serde detection for branched function components

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2843
2023-11-13 13:36:51 -05:00
Soby Chacko
d130b46110 Test cleanup in Kafka binder modules
Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 19:35:07 -05:00
Soby Chacko
71e095343c Test cleanup in Kafka Streams binder modules
Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 17:04:33 -05:00
Soby Chacko
bafffad0f5 Test cleanup in rabbit binder modules
- Remove the test name prefix from the test methods in the rabbit binder modules
 - package protected tests in rabbit binder

Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 12:49:00 -05:00
Soby Chacko
a49440d1f7 Test cleanup in schema registry modules
- Remove the test name prefix from the test methods in the
   schema-registry releated modules

Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 12:33:41 -05:00
Soby Chacko
9ebae77b1c Test cleanup in spring-cloud-stream continued
- Remove the test name prefix from the test methods in core
   spring-cloud-stream-integration-tests module

 Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 12:17:45 -05:00
Soby Chacko
849d1c669f Test cleanup in spring-cloud-stream module
- Remove the test name prefix from the test methods in core spring-cloud-stream module

Part of fixing https://github.com/spring-cloud/spring-cloud-stream/issues/2371.
2023-11-08 11:48:55 -05:00
Soby Chacko
92324641ef GH-2846: Regression issue in Kafka Streams binder
- When @Configuration or @Component style beans are used in Kafka Streams
   binder applications, there exists a regression that prevents the binder
   infrastructure to correctly detect the beans. Addressing this issue.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2846
2023-11-07 16:32:35 -05:00
Soby Chacko
b8e4885d21 Kafka Streams binder native changes
- RocksDBException and Status need INVOKE_DECLARED_CONSTRUCTOR privileges
2023-11-03 14:37:10 -04:00
Soby Chacko
36a43be235 package protected tests in rabbit binder 2023-11-02 15:34:17 -04:00
Soby Chacko
462d42b3ad package protected test classes - Pulsar binder 2023-11-02 15:10:03 -04:00
Soby Chacko
4b67440c27 package protected tests - schema registry 2023-11-02 14:58:52 -04:00
Soby Chacko
74c9e4ab80 Package protected tests in core modules 2023-11-02 14:50:10 -04:00
Soby Chacko
116d8dc535 [Test Cleanup - Package Private]
- spring-cloud-stream-binder-kafka-streams module tests
2023-11-02 14:29:25 -04:00
Soby Chacko
6af434c18a [Test Cleanup - Package Private]
- spring-cloud-stream-binder-kafka-reactive module tests
2023-11-01 15:44:21 -04:00
Soby Chacko
515923266e [Test Cleanup - Package Private]
- spring-cloud-stream-binder-kafka-core module tests
2023-11-01 15:38:07 -04:00
Soby Chacko
60dfce7adc [Test Cleanup] Kafka binder tests package private 2023-11-01 15:32:28 -04:00
Soby Chacko
903534085c Test cleanup: schema registry tests
- Remove the superfluous ExtendWith for SpringExtension
2023-11-01 14:50:12 -04:00
buildmaster
9054367f67 Going back to snapshots 2023-10-31 13:34:17 +00:00
buildmaster
b3a3e03634 Update SNAPSHOT to 4.1.0-RC1 2023-10-31 13:30:45 +00:00
Soby Chacko
c994dd5ab1 checkstyle fixes 2023-10-27 16:12:31 -04:00
Soby Chacko
a2d4bddf08 Test changes in BinderChildContextInitializer
- In child test contexts, we are not able to set the logging levels
   and this affects certain AOT integration tests. This does not seem
   to be an issue in real applications. Adjust the logging levels
   becasue of this in BinderChildContextInitializer and related classes.

 - Re-enable BinderChildContextInitializerTests
2023-10-27 15:44:46 -04:00
Soby Chacko
0603500cfc Remove unnecessary ExtendWith in core tests 2023-10-26 18:33:34 -04:00
Soby Chacko
503da99b19 Test cleanup in Kafka binder
- Remove the unncessary usage of @ExtendWith in Kafka binder tests
2023-10-26 17:55:25 -04:00
Soby Chacko
9b2e91dc60 Kafka Streams binder docs cleanup
- Use StreamsBuilderFactoryBeanConfigurer instead of StreamsBuilderFactoryBeanCustomizer

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2837
2023-10-24 19:16:57 -04:00
Soby Chacko
e8431e4600 AOT/Native related changes in Kafka Streams binder
- Add KafkaStreams related native hints (Note: Once we add the
   Kafka Streams specific 3rd party hints to the reachability
   metadata repository, we will remove those hints from the binder)
 - Related changes in DefaultBinderFactory
 - KafkaStreamsBindableProxyFactory changes triggered by AOT/native

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2323

Addressing PR review comments
Resolves #2832
2023-10-20 16:50:47 +02:00
kurt
6af320626e StreamBridge's dynamic destinations should be closed on shutdown.
- if not, we might lose some messages on shutdown especially in case of kafka async mode producing.

Resolves #2835
2023-10-20 16:03:50 +02:00
Soby Chacko
b867311333 GH-2828: Custom binders not recognized by AOT
- When using multi-binders with custom binders and running in AOT mode,
   there is a regression that is causing some issues for propertly
   identifying the binders during the AOT phase. It forces the users
   to provide property in the form of `spring.cloud.stream.binders.binders...`
   Fixing this issue by properly binding the custom binders in BinderChildContextInitializer.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2828

Resolves #2834
2023-10-20 16:01:57 +02:00
Steven PG
4a4ce6975a Add DltAwareProcessor trace logging
Since exception is not propogated, adding a trace log allows an optional way for a developer to utilize the defaultRecoverer and still do some basic review of a given exception.
2023-10-18 22:09:48 -04:00
Soby Chacko
0133a95ebd StreamBridge tests cleanup 2023-10-18 12:32:07 -04:00
Soby Chacko
5a0183fd0d GH-2815: StreamBridge partitioning fixes
- When native encoding is used, StreamBridge is not setting
   partitioning interceptor which calls the PartitionHandler.
   This is a regression issue and adding the interceptor explicitly
   when native encoding is used.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2815
Resolves #2831
2023-10-18 15:06:20 +02:00
Soby Chacko
7f899646d3 Docs changes for the DltAwareProcessor hierarchy 2023-10-17 13:14:16 -04:00
Soby Chacko
a74ea9fe87 Introduce a hierarchy for DltAwareProcessor
- Common abstraction - RecordRecoverableProcessor which DltAwareProcessor extends
2023-10-17 13:02:55 -04:00
Soby Chacko
04e27d1bbe AOT required changes in Kafka Streams binder 2023-10-13 17:32:24 -04:00
Soby Chacko
4b53c839ac Sending record keys as part of DltAwareProcessor 2023-10-12 20:26:32 -04:00
Oleg Zhurakousky
f811ed0b7c Cleanup and dceprecation removal in core 2023-10-10 13:12:32 +02:00
Soby Chacko
d650f9e1e8 Kafka Streams binder code cleanup
- Remove deprecated API usage
 - Other cleanup
 - EmbeddedKafka changes in tests
2023-10-09 21:12:27 -04:00
dependabot[bot]
93c4fe39ae Bump org.apache.avro:avro from 1.11.1 to 1.11.3 in /schema-registry
Bumps org.apache.avro:avro from 1.11.1 to 1.11.3.

---
updated-dependencies:
- dependency-name: org.apache.avro:avro
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-10-02 10:05:10 -04:00
Soby Chacko
cf6cea6525 GH-2817: Method name clash in Kafka Streams binder
- When there are two methods with the same name but with different type erasures,
   Kafka Streams binder sometimes detects the incorrect method. Fixing this issue
   by specifically type checking the return type for Kafka Streams types.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2817
2023-09-29 21:24:31 -04:00
Soby Chacko
d1a0aff623 GH-2820: Component beans and function return types
- When Kafka Streams functions are provided as component beans,
   the function type check in the core Spring Cloud Stream framework,
   fails with an NPE. Addressing this issue.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2820
2023-09-29 17:00:09 -04:00
Soby Chacko
3a52c3b3cf GH-2821: Kafka Streams branching as Component bean
- When Kafka Streams branching function is provided as a Component bean,
   there is an issue where the raw class check on the return outbound
   resolvable type cuases an NPE. Fixing this issue by adding a null check
   on the return type's raw class.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2821
2023-09-29 16:27:04 -04:00
Soby Chacko
895570fb48 Code cleanup: AbstractKafkaStreamsBinderProcessor
- Remove deprecation warnings
 - Other code cleanup
2023-09-29 15:33:09 -04:00
Soby Chacko
5ea391beb0 GH-2823: Messages routed to the wrong consumer
- When event-type routing is enabled in Kafka Streams binder
   and conurrency > 1 is used, messages are occasionally getting
   dispatched to the wrong consumer causing CCE. This is due
   to a race condition caused by a shared resource across threads.
   Fixing the issue by introducing a ThreadLocal variable.

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/2823
2023-09-29 10:43:43 -04:00