-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
CAMEL-20010 #11759
CAMEL-20010 #11759
Conversation
…oid replacing the table name more than once
🌟 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:
|
...el-sql/src/main/java/org/apache/camel/processor/idempotent/jdbc/JdbcMessageIdRepository.java
Outdated
Show resolved
Hide resolved
…essary usage of regular expressions
I'm wondering if it is normal that the |
doInit ensures its only invoked once |
* CAMEL-20010 Preserve default queries in JdbcMessageIdRepository to avoid replacing the table name more than once * CAMEL-20010 Use String#replace instead of replaceFirst to avoid unnecessary usage of regular expressions --------- Co-authored-by: Bruno Mendola <bruno.mendola@dedagroup.it>
* CAMEL-20010 Preserve default queries in JdbcMessageIdRepository to avoid replacing the table name more than once * CAMEL-20010 Use String#replace instead of replaceFirst to avoid unnecessary usage of regular expressions --------- Co-authored-by: Bruno Mendola <bruno.mendola@dedagroup.it>
* CAMEL-20010 Preserve default queries in JdbcMessageIdRepository to avoid replacing the table name more than once * CAMEL-20010 Use String#replace instead of replaceFirst to avoid unnecessary usage of regular expressions --------- Co-authored-by: Bruno Mendola <bruno.mendola@dedagroup.it>
* CAMEL-20010 Preserve default queries in JdbcMessageIdRepository to avoid replacing the table name more than once * CAMEL-20010 Use String#replace instead of replaceFirst to avoid unnecessary usage of regular expressions --------- Co-authored-by: Bruno Mendola <bruno.mendola@dedagroup.it>
So why do we get what is described in the ticket? I quote:
If I understand it well, if |
...
I think @essobedo has a point. I mean... the replacement was clearly happening twice, but the Maybe it's something related to some particular condition? |
Description
Preserve default queries in
JdbcMessageIdRepository
to avoid replacing the table name more than once.Solves CAMEL-20010.
Target
camel-3.x
, whereas Camel 4 uses themain
branch)Tracking
Apache Camel coding standards and style
mvn clean install -DskipTests
locally and I have committed all auto-generated changes