-
Enhancement
-
Resolution: Done
-
Major
-
None
-
None
-
None
-
NEW
-
NEW
...ore/src/main/java/org/optaplanner/core/impl/score/buildin/bendable/BendableScoreInliner.java
if (constraintMatchEnabled) { matchScoreConsumer.accept(BendableScore.ofHard(hardScores.length, softScores.length, level, hardImpact)); } return () -> this.hardScores[level] -= hardImpact;
This last statement is hard to read.
My main concern is that constraint match totals got bolted on, causing the ScoreImpacter to have a matchScoreConsumer - this had a noticeable performance impact even when constraint matches are disabled. I suspect much of this code will change when I take a second stab at it. Right now, the behaviour is correct - but it's too complex, it needs to be simplified.
- relates to
-
PLANNER-1584 ConstraintStreams loose ends
-
- Resolved
-