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.

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

Support camel 3.0.0 M3 #79 #81

Merged
merged 1 commit into from
Jun 10, 2019

Conversation

lburgazzoli
Copy link
Contributor

No description provided.

Copy link
Member

@valdar valdar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

import java.util.List;
import java.util.Properties;

import org.apache.camel.CamelContext;
import org.apache.camel.Ordered;
import org.apache.camel.component.properties.PropertiesComponent;
import org.apache.camel.impl.DefaultCamelContext;
import org.apache.camel.impl.ExplicitCamelContextNameStrategy;
import org.apache.camel.impl.engine.ExplicitCamelContextNameStrategy;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you need this one for being able to set the name easily, then the setName api is now public. But I think this may be from M4 onwards

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup, will clean up once M4 will be out out

}

public static int bindProperties(Properties properties, Object target, String prefix) {
public static int bindProperties(CamelContext context, Properties properties, Object target, String prefix) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could likely add API on PropertyBindingSupport to support prefix so we have it OOTB, fell free to log a JIRA

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes I'm working on this

@lburgazzoli lburgazzoli merged commit 88be1e1 into apache:master Jun 10, 2019
@lburgazzoli lburgazzoli deleted the github-79-camel-3.0.0-M3 branch June 10, 2019 18:47
squakez pushed a commit to squakez/camel-k-runtime that referenced this pull request Mar 27, 2023
…pache#81)

PNC build is failing because it is trying to use a productized version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants