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
OptaPlanner native support fixes #1721 #1822
Conversation
import org.optaplanner.core.api.domain.lookup.PlanningId; | ||
|
||
/** | ||
* adapted from optaplanner quarksu quickstart : |
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.
quarksu
-> quarkus
(same for the other places where this is mentioned).
Wondering what this means for future CQ releases. I guess we may need to wait for OptaPlanner to bump their bits to the latest Quarkus version before we can proceed with our release? |
309081c
to
aae8afd
Compare
@davsclaus @jamesnetherton : I did some changes |
fdd43ac
to
d8128f5
Compare
a26946a
to
4788828
Compare
@zbendhiba could you please rebase and solve the conflicts? I'd like to review locally, esp. the dependency management part |
aae8afd
to
bf508d8
Compare
@ppalaga I rebased, but please do not merge. The tests will fail with graalvm 20.2.0. We should wait for an update of version of extension |
b515430
to
d13f6cc
Compare
I'll set this PR to draft until we have everything in place to merge. |
d470e69
to
9253684
Compare
9253684
to
2628b89
Compare
8e644d2
to
3b7ae6e
Compare
3b7ae6e
to
5670648
Compare
ed1eb3f
to
9715193
Compare
9715193
to
9c131ab
Compare
9c131ab
to
d40d9c1
Compare
d8d9fc9
to
76c1520
Compare
76c1520
to
b80eca8
Compare
Could you please resolve the conflicts, @zbendhiba ? |
@ppalaga yes |
b80eca8
to
8999db2
Compare
native support of optaplanner using optaplanner quarkus extension and recent support of SolverManager in Camel.