[GitHub] [camel] oscerd commented on pull request #10733: integrate multi-shard consumer with async client

2023-07-19 Thread via GitHub


oscerd commented on PR #10733:
URL: https://github.com/apache/camel/pull/10733#issuecomment-1642302225

   Test is failing:
   
   ```
   [INFO] Using the SmartBuilder implementation with a thread count of 2
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] < 
org.apache.camel:camel-aws2-kinesis >-
   [camel-aws2-kinesis] [INFO] Building Camel :: AWS2 Kinesis 4.0.0-SNAPSHOT
 [1/28]
   [camel-aws2-kinesis] [INFO]   from pom.xml
   [camel-aws2-kinesis] [INFO] [ jar 
]-
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- build-helper:3.4.0:add-source (default) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Source directory: 
/home/runner/work/camel/camel/components/camel-aws/camel-aws2-kinesis/src/generated/java
 added.
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- build-helper:3.4.0:add-resource (default) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- license:4.2:format (license-format) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [WARN] Parameter 'legacyConfigExcludes' (user property 
'license.excludes') is deprecated: use LicenseSet.excludes
   [camel-aws2-kinesis] [WARN] Parameter 'legacyConfigHeader' (user property 
'license.header') is deprecated: use LicenseSet.header
   [camel-aws2-kinesis] [INFO] Updating license headers...
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- formatter:2.23.0:format (format) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Processed 51 files in 1s735ms (Formatted: 1, 
Skipped: 0, Unchanged: 50, Failed: 0, Readonly: 0)
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- impsort:1.9.0:sort (sort-imports) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Processed 51 files in 00:00.590 (Already Sorted: 
50, Needed Sorting: 1)
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- resources:3.3.1:resources 
(default-resources) @ camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] skip non existing resourceDirectory 
/home/runner/work/camel/camel/components/camel-aws/camel-aws2-kinesis/src/main/resources
   [camel-aws2-kinesis] [INFO] Copying 13 resources from 
src/generated/resources to target/classes
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- flatten:1.5.0:flatten (default-cli) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Generating flattened POM of project 
org.apache.camel:camel-aws2-kinesis:jar:4.0.0-SNAPSHOT...
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- compiler:3.11.0:compile (default-compile) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Changes detected - recompiling the module! 
:source
   [camel-aws2-kinesis] [INFO] Compiling 41 source files with javac [debug 
deprecation release 17] to target/classes
   [camel-aws2-kinesis] [INFO] 
/home/runner/work/camel/camel/components/camel-aws/camel-aws2-kinesis/src/main/java/org/apache/camel/component/aws2/firehose/KinesisFirehose2Producer.java:
 Some input files use unchecked or unsafe operations.
   [camel-aws2-kinesis] [INFO] 
/home/runner/work/camel/camel/components/camel-aws/camel-aws2-kinesis/src/main/java/org/apache/camel/component/aws2/firehose/KinesisFirehose2Producer.java:
 Recompile with -Xlint:unchecked for details.
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- camel-package:4.0.0-SNAPSHOT:generate 
(generate) @ camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Stale files detected, re-generating.
   [camel-aws2-kinesis] [INFO] Stale files detected, re-generating index.
   [camel-aws2-kinesis] [INFO] Building index...
   [camel-aws2-kinesis] [INFO] Generated components containing 2 Camel 
components: aws2-kinesis aws2-kinesis-firehose
   [camel-aws2-kinesis] [INFO] Validation complete
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- compiler:3.11.0:compile (recompile) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Nothing to compile - all classes are up to date
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- resources:3.3.1:testResources 
(default-testResources) @ camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Copying 1 resource from src/test/resources to 
target/test-classes
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- compiler:3.11.0:testCompile 
(default-testCompile) @ camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Changes detected - recompiling the module! 
:dependency
   [camel-aws2-kinesis] [INFO] Compiling 17 source files with javac [debug 
deprecation release 17] to target/test-classes
   [camel-aws2-kinesis] [INFO] 
   [camel-aws2-kinesis] [INFO] --- surefire:3.1.2:test (default-test) @ 
camel-aws2-kinesis ---
   [camel-aws2-kinesis] [INFO] Surefire report directory: 
/home/runner/work/camel/camel/components/camel-aws/camel-aws2-kinesis/target/

[GitHub] [camel] oscerd commented on pull request #10733: integrate multi-shard consumer with async client

2023-07-19 Thread via GitHub


oscerd commented on PR #10733:
URL: https://github.com/apache/camel/pull/10733#issuecomment-1642501070

   @orpiske @davsclaus We need to understand if we could have both the async 
and sync in the same module, or we need to revert and split.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org