Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add camel-python language extension #6328

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add camel-python language extension #6328

wants to merge 1 commit into from

Conversation

kulagaIA
Copy link
Member

@kulagaIA kulagaIA commented Aug 8, 2024 •
edited
Loading

Added camel-quarkus-python extension that depends on camel version 4.8.0

Issue:
#4408

@zhfeng
Copy link
Contributor

zhfeng commented Aug 9, 2024

@kulagaIA in this case, can you try to target against with camel-main branch?

@kulagaIA kulagaIA changed the base branch from main to camel-main August 9, 2024 07:23
@kulagaIA kulagaIA force-pushed the python branch 9 times, most recently from 1450889 to 4414a30 Compare August 12, 2024 12:46
@kulagaIA kulagaIA force-pushed the python branch 3 times, most recently from 271a54e to 1588af5 Compare August 13, 2024 14:26
@kulagaIA kulagaIA marked this pull request as ready for review August 13, 2024 14:34
@kulagaIA kulagaIA requested a review from zhfeng August 13, 2024 14:35
@jamesnetherton
Copy link
Contributor

Might be better to wait for Camel 4.8.0 to be integrated on the main branch. You'll be starting from a stable baseline.

camel-main is a constantly moving target unfortunately.

@kulagaIA kulagaIA force-pushed the python branch 4 times, most recently from 68c7eab to 6a264f8 Compare August 16, 2024 11:55
@JiriOndrusek JiriOndrusek force-pushed the camel-main branch 2 times, most recently from 387462a to ffaea8d Compare August 26, 2024 07:37
@github-actions github-actions bot force-pushed the camel-main branch 3 times, most recently from a9dd3e0 to 721293b Compare September 9, 2024 01:50
@jamesnetherton
Copy link
Contributor

@kulagaIA please recreate this against the main branch.

You'll need to update the version on the modules you added to 3.15.0-SNAPSHOT. Also update the camel.quarkus.jvmSince & camel.quarkus.nativeSince properties in runtime/pom.xml to 3.15.0.

@jamesnetherton jamesnetherton changed the base branch from camel-main to main September 18, 2024 06:52
@jamesnetherton
Copy link
Contributor

I tidied up the branch and got the tests running in JVM mode.

For native mode, there's an issue at build time. Probably something in PythonLanguage or Jython itself cannot be serialized properly.

[error]: Build step io.quarkus.deployment.steps.MainClassBuildStep#build threw an exception: java.lang.StackOverflowError
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadComplexObject(BytecodeRecorderImpl.java:1122)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstanceImpl(BytecodeRecorderImpl.java:914)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstance(BytecodeRecorderImpl.java:586)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadComplexObject(BytecodeRecorderImpl.java:1361)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstanceImpl(BytecodeRecorderImpl.java:914)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadObjectInstance(BytecodeRecorderImpl.java:586)
    at io.quarkus.deployment.recording.BytecodeRecorderImpl.loadComplexObject(BytecodeRecorderImpl.java:1361)

@jamesnetherton jamesnetherton added the do-not-merge/hold Indicates that a PR should not be merged because there are unresolved issues pending label Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhfeng zhfeng Awaiting requested review from zhfeng

Assignees
No one assigned
Labels
do-not-merge/hold Indicates that a PR should not be merged because there are unresolved issues pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kulagaIA @zhfeng @jamesnetherton