-
Notifications
You must be signed in to change notification settings - Fork 5.1k
CAMEL-22222: Add component for Apache Iggy #18583
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🤖 CI automation will test this PR automatically. 🐫 Apache Camel Committers, please review the following items:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great start.
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyConfiguration.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyConfiguration.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyConfiguration.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyConfiguration.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyEndpoint.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyProducer.java
Outdated
Show resolved
Hide resolved
components/camel-iggy/src/main/java/org/apache/camel/component/iggy/IggyProducer.java
Outdated
Show resolved
Hide resolved
With the last commit, we are now in parity with the iggy-java-sdk feature-wise. I opened a Jira issue for future development. |
The failure on the gh actioin is expected, it is due to |
Also remember to add a CSB starter for this |
has iggy been released |
Not yet, I am monitoring https://repo1.maven.org/maven2/org/apache/iggy/iggy-java-sdk once a day |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one!
iggy 0.5.0 java SDKs are released |
@davsclaus @oscerd Hi, the released iggy-java-sdk 0.5.0 was built with JDK21, therefore, the build with JDK17 is failing https://repo1.maven.org/maven2/org/apache/iggy/iggy-java-sdk/0.5.0/ The error is |
Ah yeah after the 4.14 LTS then there is maybe a chance to drop JDK17 when we jump to SB 4 and so on. Also JDK25 will be released, so that is also a good opportunity as well. |
Ok, I'll put the PR back in draft, to be merged once JDK 17 will be dropped and Apache Camel JDK basline will be JDK 21 (after 4.14 release most probably) |
I am waiting for the iggy java client to be released upstream, it should be released in a week or so, right now the development version 0.2.0-SNAPSHOT is being used.
Supported functionalities:
The implemented functionalities are basic, and are based on the java examples provided by Iggy, I'll have to investigate more to implement other usecases.