This is an automated email from the ASF dual-hosted git repository.
mariofusco pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-kie-drools.git
The following commit(s) were added to refs/heads/main by this push:
new 81dce55001 [incubator-kie-drools-6442] Remove reteevaluator from mvel
(#6451)
81dce55001 is described below
commit 81dce5500156b851fc550f0931898a8bb74b8682
Author: Paolo Bizzarri <[email protected]>
AuthorDate: Thu Sep 11 08:24:05 2025 +0200
[incubator-kie-drools-6442] Remove reteevaluator from mvel (#6451)
* Removing ReteEvaluator from mvel
* Removed deps of core from drools-mvel
* More cleanups
---
.../main/java/org/drools/core/phreak/PhreakAccumulateNode.java | 3 ++-
.../src/main/java/org/drools/core/reteoo/AccumulateNode.java | 5 +++--
.../src/main/java/org/drools/mvel/EvaluatorConstraint.java | 3 +--
drools-mvel/src/main/java/org/drools/mvel/EvaluatorHelper.java | 3 +--
drools-mvel/src/main/java/org/drools/mvel/MVELConstraint.java | 8 +++-----
.../src/main/java/org/drools/mvel/MVELConstraintBuilder.java | 7 ++-----
.../src/main/java/org/drools/mvel/MVELGroupByAccumulate.java | 5 ++---
.../main/java/org/drools/mvel/asm/ClassFieldAccessorFactory.java | 3 +--
.../src/main/java/org/drools/mvel/asm/GeneratorHelper.java | 3 +--
.../java/org/drools/mvel/evaluators/AfterEvaluatorDefinition.java | 6 +++---
.../main/java/org/drools/mvel/evaluators/VariableRestriction.java | 3 +--
.../src/main/java/org/drools/mvel/expr/MVELAccumulator.java | 3 +--
.../src/main/java/org/drools/mvel/expr/MVELConsequence.java | 3 +--
.../src/main/java/org/drools/mvel/expr/MVELEnabledExpression.java | 3 +--
.../src/main/java/org/drools/mvel/expr/MVELEvalExpression.java | 3 +--
.../src/main/java/org/drools/mvel/expr/MVELObjectExpression.java | 3 +--
.../main/java/org/drools/mvel/expr/MVELReturnValueExpression.java | 5 +----
.../main/java/org/drools/mvel/expr/MVELSalienceExpression.java | 3 +--
.../src/main/java/org/drools/mvel/field/ClassFieldImpl.java | 1 -
19 files changed, 27 insertions(+), 46 deletions(-)
diff --git
a/drools-core/src/main/java/org/drools/core/phreak/PhreakAccumulateNode.java
b/drools-core/src/main/java/org/drools/core/phreak/PhreakAccumulateNode.java
index 63bc955189..3b50ac6dcc 100644
--- a/drools-core/src/main/java/org/drools/core/phreak/PhreakAccumulateNode.java
+++ b/drools-core/src/main/java/org/drools/core/phreak/PhreakAccumulateNode.java
@@ -18,6 +18,7 @@
*/
package org.drools.core.phreak;
+import org.drools.base.base.ValueResolver;
import org.drools.base.reteoo.AccumulateContextEntry;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Accumulate;
@@ -206,7 +207,7 @@ public class PhreakAccumulateNode {
return accContext;
}
- public static void initContext(Object workingMemoryContext, ReteEvaluator
reteEvaluator, Accumulate accumulate, BaseTuple leftTuple,
AccumulateContextEntry accContext) {
+ public static void initContext(Object workingMemoryContext, ValueResolver
reteEvaluator, Accumulate accumulate, BaseTuple leftTuple,
AccumulateContextEntry accContext) {
// Create the function context, but allow init to override it.
Object funcContext = accumulate.createFunctionContext();
funcContext = accumulate.init(workingMemoryContext, accContext,
funcContext, leftTuple, reteEvaluator);
diff --git
a/drools-core/src/main/java/org/drools/core/reteoo/AccumulateNode.java
b/drools-core/src/main/java/org/drools/core/reteoo/AccumulateNode.java
index f57ee47aac..c6dbac5801 100644
--- a/drools-core/src/main/java/org/drools/core/reteoo/AccumulateNode.java
+++ b/drools-core/src/main/java/org/drools/core/reteoo/AccumulateNode.java
@@ -25,6 +25,7 @@ import java.util.Map;
import org.drools.base.base.ClassObjectType;
import org.drools.base.base.ObjectType;
+import org.drools.base.base.ValueResolver;
import org.drools.base.common.NetworkNode;
import org.drools.base.reteoo.AccumulateContextEntry;
import org.drools.base.reteoo.BaseTuple;
@@ -184,7 +185,7 @@ public class AccumulateNode extends BetaNode {
* Creates a BetaMemory for the BetaNode's memory.
*/
public Memory createMemory(final RuleBaseConfiguration config,
ReteEvaluator reteEvaluator) {
- BetaMemory betaMemory = (BetaMemory)
this.constraints.createBetaMemory(config, NodeTypeEnums.AccumulateNode);
+ BetaMemory betaMemory = this.constraints.createBetaMemory(config,
NodeTypeEnums.AccumulateNode);
AccumulateMemory memory = this.accumulate.isMultiFunction() ?
new MultiAccumulateMemory(betaMemory,
this.accumulate.getAccumulators()) :
new SingleAccumulateMemory(betaMemory,
this.accumulate.getAccumulators()[0]);
@@ -320,7 +321,7 @@ public class AccumulateNode extends BetaNode {
}
public TupleListWithContext<AccumulateContextEntry> getGroup(Object
workingMemoryContext, Accumulate accumulate, BaseTuple leftTuple,
- Object key,
ReteEvaluator reteEvaluator) {
+ Object key,
ValueResolver reteEvaluator) {
return groupsMap.computeIfAbsent(key, k -> {
AccumulateContextEntry entry = new AccumulateContextEntry(key);
entry.setFunctionContext(
accumulate.init(workingMemoryContext, entry,
accumulate.createFunctionContext(), leftTuple, reteEvaluator) );
diff --git a/drools-mvel/src/main/java/org/drools/mvel/EvaluatorConstraint.java
b/drools-mvel/src/main/java/org/drools/mvel/EvaluatorConstraint.java
index 36e6cdda3d..cea1075478 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/EvaluatorConstraint.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/EvaluatorConstraint.java
@@ -24,7 +24,6 @@ import java.io.ObjectOutput;
import java.util.Arrays;
import org.drools.base.base.ValueResolver;
-import org.drools.core.common.InternalFactHandle;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
import org.drools.base.rule.IntervalProviderConstraint;
@@ -187,7 +186,7 @@ public class EvaluatorConstraint extends
MutableTypeConstraint<ContextEntry> imp
public void readExternal(ObjectInput in) throws IOException,
ClassNotFoundException {
extractor = (ReadAccessor) in.readObject();
- factHandle = ( InternalFactHandle ) in.readObject();
+ factHandle = ( FactHandle ) in.readObject();
next = (ContextEntry) in.readObject();
valueResolver = ( ValueResolver ) in .readObject();
}
diff --git a/drools-mvel/src/main/java/org/drools/mvel/EvaluatorHelper.java
b/drools-mvel/src/main/java/org/drools/mvel/EvaluatorHelper.java
index d91cc575a6..30958340b7 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/EvaluatorHelper.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/EvaluatorHelper.java
@@ -27,7 +27,6 @@ import org.drools.base.base.CoreComponentsBuilder;
import org.drools.compiler.rule.builder.EvaluatorWrapper;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
-import org.drools.core.reteoo.Tuple;
import org.kie.api.runtime.rule.FactHandle;
public class EvaluatorHelper {
@@ -50,7 +49,7 @@ public class EvaluatorHelper {
return map;
}
- public static void initOperators(FactHandle handle, Tuple tuple,
EvaluatorWrapper[] operators) {
+ public static void initOperators(FactHandle handle, BaseTuple tuple,
EvaluatorWrapper[] operators) {
FactHandle[] handles = tuple != null ? tuple.toFactHandles() : new
FactHandle[0];
for (EvaluatorWrapper operator : operators) {
operator.loadHandles(handles, handle);
diff --git a/drools-mvel/src/main/java/org/drools/mvel/MVELConstraint.java
b/drools-mvel/src/main/java/org/drools/mvel/MVELConstraint.java
index 76a67a5e7e..3871c6ba7c 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/MVELConstraint.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/MVELConstraint.java
@@ -61,8 +61,6 @@ import org.drools.base.rule.constraint.Constraint;
import org.drools.base.util.IndexedValueReader;
import org.drools.base.util.index.ConstraintTypeOperator;
import org.drools.compiler.rule.builder.EvaluatorWrapper;
-import org.drools.core.impl.KnowledgeBaseImpl;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.ConditionAnalyzer.CombinedCondition;
import org.drools.mvel.ConditionAnalyzer.Condition;
import org.drools.mvel.ConditionAnalyzer.EvaluatedExpression;
@@ -812,8 +810,8 @@ public class MVELConstraint extends
MutableTypeConstraint<ContextEntry> implemen
return true;
}
- Map<String, Object> thisImports = (( MVELDialectRuntimeData )
((KnowledgeBaseImpl)kbase).getPackage(thisPkg).getDialectRuntimeRegistry().getDialectData("mvel")).getImports();
- Map<String, Object> otherImports = (( MVELDialectRuntimeData )
((KnowledgeBaseImpl)kbase).getPackage(otherPkg).getDialectRuntimeRegistry().getDialectData("mvel")).getImports();
+ Map<String, Object> thisImports = (( MVELDialectRuntimeData )
kbase.getPackage(thisPkg).getDialectRuntimeRegistry().getDialectData("mvel")).getImports();
+ Map<String, Object> otherImports = (( MVELDialectRuntimeData )
kbase.getPackage(otherPkg).getDialectRuntimeRegistry().getDialectData("mvel")).getImports();
if (fieldValue != null && constraintType.getOperator() !=
null) {
return
equalsExpressionTokensInBothImports(getLeftInExpression(constraintType),
thisImports, otherImports);
@@ -877,7 +875,7 @@ public class MVELConstraint extends
MutableTypeConstraint<ContextEntry> implemen
protected MVELDialectRuntimeData getMVELDialectRuntimeData(RuleBase kbase)
{
for (String packageName : packageNames) {
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)kbase).getPackage(packageName);
+ InternalKnowledgePackage pkg = kbase.getPackage(packageName);
if (pkg != null) {
return ((MVELDialectRuntimeData)
pkg.getDialectRuntimeRegistry().getDialectData("mvel"));
}
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/MVELConstraintBuilder.java
b/drools-mvel/src/main/java/org/drools/mvel/MVELConstraintBuilder.java
index 4cf7cf787a..abe4a51cd0 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/MVELConstraintBuilder.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/MVELConstraintBuilder.java
@@ -21,7 +21,6 @@ package org.drools.mvel;
import java.io.IOException;
import java.io.ObjectInput;
import java.io.ObjectOutput;
-import java.lang.reflect.Modifier;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
@@ -30,7 +29,6 @@ import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
-import java.util.Objects;
import java.util.Set;
import org.drools.base.base.DroolsQuery;
@@ -118,7 +116,6 @@ import static
org.drools.compiler.rule.builder.PatternBuilder.registerDescrBuild
import static
org.drools.compiler.rule.builder.util.PatternBuilderUtil.getNormalizeDate;
import static
org.drools.compiler.rule.builder.util.PatternBuilderUtil.normalizeEmptyKeyword;
import static
org.drools.compiler.rule.builder.util.PatternBuilderUtil.normalizeStringOperator;
-import static org.drools.util.ClassUtils.convertFromPrimitiveType;
import static org.drools.mvel.asm.AsmUtil.copyErrorLocation;
import static org.drools.mvel.builder.MVELExprAnalyzer.analyze;
import static org.drools.mvel.expr.MvelEvaluator.createMvelEvaluator;
@@ -545,7 +542,7 @@ public class MVELConstraintBuilder implements
ConstraintBuilder {
private static MVELAnalysisResult analyzeExpression(String expr,
ParserConfiguration conf, BoundIdentifiers availableIdentifiers) {
if (expr.trim().isEmpty()) {
- MVELAnalysisResult result = analyze( (Set<String> )
Collections.EMPTY_SET, availableIdentifiers );
+ MVELAnalysisResult result = analyze( Collections.emptySet(),
availableIdentifiers );
result.setMvelVariables( new HashMap<>() );
result.setTypesafe( true );
return result;
@@ -628,7 +625,7 @@ public class MVELConstraintBuilder implements
ConstraintBuilder {
MVELAnalysisResult result = analyze( requiredInputs,
availableIdentifiers );
result.setReturnType( returnType );
- result.setMvelVariables( (Map<String, Class< ? >>) (Map) variables );
+ result.setMvelVariables( (Map) variables );
result.setTypesafe( true );
return result;
}
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/MVELGroupByAccumulate.java
b/drools-mvel/src/main/java/org/drools/mvel/MVELGroupByAccumulate.java
index b737fd05c6..b57b4deda2 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/MVELGroupByAccumulate.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/MVELGroupByAccumulate.java
@@ -28,7 +28,6 @@ import org.drools.base.rule.accessor.FieldValue;
import org.drools.base.rule.accessor.ReturnValueExpression;
import org.drools.base.rule.accessor.Wireable;
import org.drools.core.common.InternalFactHandle;
-import org.drools.core.common.ReteEvaluator;
import org.drools.core.reteoo.AccumulateNode;
import org.drools.core.reteoo.EvalNodeLeftTuple;
import org.drools.core.reteoo.Tuple;
@@ -61,7 +60,7 @@ public class MVELGroupByAccumulate extends Accumulate {
return innerAccumulate;
}
- private Object getKey( Tuple tuple, FactHandle handle, ReteEvaluator
reteEvaluator ) {
+ private Object getKey( Tuple tuple, FactHandle handle, ValueResolver
reteEvaluator ) {
try {
Tuple keyTuple = isMvel? tuple : new
EvalNodeLeftTuple((InternalFactHandle) handle, (TupleImpl) tuple, tuple
.getSink());
@@ -114,7 +113,7 @@ public class MVELGroupByAccumulate extends Accumulate {
BaseTuple match, FactHandle handle, ValueResolver valueResolver) {
AccumulateNode.GroupByContext groupByContext =
(AccumulateNode.GroupByContext) context;
TupleListWithContext<AccumulateContextEntry> tupleList =
groupByContext.getGroup(workingMemoryContext, innerAccumulate,
-
match, getKey( (Tuple) match, handle, (ReteEvaluator) valueResolver),
(ReteEvaluator) valueResolver);
+
match, getKey( (Tuple) match, handle, valueResolver), valueResolver);
return accumulate(workingMemoryContext, match, handle, groupByContext,
tupleList, valueResolver);
}
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/asm/ClassFieldAccessorFactory.java
b/drools-mvel/src/main/java/org/drools/mvel/asm/ClassFieldAccessorFactory.java
index 151b0f5918..fc990a60a7 100644
---
a/drools-mvel/src/main/java/org/drools/mvel/asm/ClassFieldAccessorFactory.java
+++
b/drools-mvel/src/main/java/org/drools/mvel/asm/ClassFieldAccessorFactory.java
@@ -26,7 +26,6 @@ import org.drools.core.base.*;
import org.drools.core.base.ClassFieldAccessorCache.CacheEntry;
import org.drools.base.base.extractors.BaseObjectClassFieldReader;
import org.drools.base.base.extractors.SelfReferenceClassFieldReader;
-import org.drools.core.common.InternalWorkingMemory;
import org.drools.base.util.Drools;
import org.drools.mvel.accessors.*;
import org.drools.wiring.api.util.ByteArrayClassLoader;
@@ -358,7 +357,7 @@ public class ClassFieldAccessorFactory implements
FieldAccessorFactory {
l1,
0 );
mv.visitLocalVariable( "workingMemory",
- Type.getDescriptor( InternalWorkingMemory.class
),
+ Type.getDescriptor( ValueResolver.class ),
null,
l0,
l1,
diff --git a/drools-mvel/src/main/java/org/drools/mvel/asm/GeneratorHelper.java
b/drools-mvel/src/main/java/org/drools/mvel/asm/GeneratorHelper.java
index 26c7d9c5e5..51fb6a2539 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/asm/GeneratorHelper.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/asm/GeneratorHelper.java
@@ -26,7 +26,6 @@ import java.util.Set;
import org.drools.base.base.ValueResolver;
import org.drools.base.reteoo.BaseTuple;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.util.ClassTypeResolver;
import org.drools.util.TypeResolver;
import org.drools.core.common.InternalFactHandle;
@@ -136,7 +135,7 @@ public final class GeneratorHelper {
}
static TypeResolver getTypeResolver(final InvokerStub stub, final
ValueResolver valueResolver, final ClassLoader classLoader) {
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage(stub.getPackageName());
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage(stub.getPackageName());
TypeResolver typeResolver = pkg == null ? null : pkg.getTypeResolver();
if (typeResolver == null) {
Set<String> imports = new HashSet<>();
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/evaluators/AfterEvaluatorDefinition.java
b/drools-mvel/src/main/java/org/drools/mvel/evaluators/AfterEvaluatorDefinition.java
index d45bd47ab4..307162a582 100644
---
a/drools-mvel/src/main/java/org/drools/mvel/evaluators/AfterEvaluatorDefinition.java
+++
b/drools-mvel/src/main/java/org/drools/mvel/evaluators/AfterEvaluatorDefinition.java
@@ -29,9 +29,9 @@ import org.drools.base.base.ValueType;
import org.drools.compiler.rule.builder.EvaluatorDefinition;
import org.drools.drl.parser.impl.Operator;
import org.drools.base.util.TimeIntervalParser;
-import org.drools.core.common.DefaultEventHandle;
import org.drools.base.rule.accessor.Evaluator;
import org.drools.base.time.Interval;
+import org.kie.api.runtime.rule.EventHandle;
import org.kie.api.runtime.rule.FactHandle;
/**
@@ -249,12 +249,12 @@ public class AfterEvaluatorDefinition
@Override
protected long getLeftTimestamp( FactHandle handle) {
- return ( (DefaultEventHandle) handle ).getEndTimestamp();
+ return ( (EventHandle) handle ).getEndTimestamp();
}
@Override
protected long getRightTimestamp( FactHandle handle ) {
- return ( (DefaultEventHandle) handle ).getStartTimestamp();
+ return ( (EventHandle) handle ).getStartTimestamp();
}
}
}
\ No newline at end of file
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/evaluators/VariableRestriction.java
b/drools-mvel/src/main/java/org/drools/mvel/evaluators/VariableRestriction.java
index 691671fbdd..52acd26cb9 100644
---
a/drools-mvel/src/main/java/org/drools/mvel/evaluators/VariableRestriction.java
+++
b/drools-mvel/src/main/java/org/drools/mvel/evaluators/VariableRestriction.java
@@ -34,7 +34,6 @@ import
org.drools.mvel.evaluators.MeetsEvaluatorDefinition.MeetsEvaluator;
import org.drools.mvel.evaluators.MetByEvaluatorDefinition.MetByEvaluator;
import org.drools.core.common.DefaultEventHandle;
import org.drools.base.rule.accessor.Evaluator;
-import org.drools.core.reteoo.Tuple;
import org.kie.api.runtime.rule.EventHandle;
import org.kie.api.runtime.rule.FactHandle;
@@ -122,7 +121,7 @@ public class VariableRestriction {
evaluator = (Evaluator) in.readObject();
object = in.readObject();
declaration = (Declaration) in.readObject();
- tuple = (Tuple) in.readObject();
+ tuple = (BaseTuple) in.readObject();
entry = (ContextEntry) in.readObject();
leftNull = in.readBoolean();
rightNull = in.readBoolean();
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELAccumulator.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELAccumulator.java
index 0c2317519e..3d9485a186 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELAccumulator.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELAccumulator.java
@@ -33,7 +33,6 @@ import org.drools.base.definitions.rule.impl.RuleImpl;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
import org.drools.base.rule.accessor.Accumulator;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.drools.mvel.expr.MVELCompilationUnit.DroolsVarFactory;
import org.kie.api.runtime.rule.FactHandle;
@@ -123,7 +122,7 @@ public class MVELAccumulator
VariableResolverFactory factory = factoryContext.getInitFactory();
initUnit.updateFactory( null, tuple, localVars, valueResolver,
valueResolver.getGlobalResolver(), factory );
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( "mvel" );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELConsequence.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELConsequence.java
index e767e66fcd..8b7378cbd5 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELConsequence.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELConsequence.java
@@ -29,7 +29,6 @@ import org.drools.base.definitions.InternalKnowledgePackage;
import org.drools.base.definitions.rule.impl.RuleImpl;
import org.drools.base.rule.consequence.Consequence;
import org.drools.core.rule.consequence.KnowledgeHelper;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.mvel2.integration.VariableResolverFactory;
@@ -88,7 +87,7 @@ public class MVELConsequence
knowledgeHelper.getRule(), knowledgeHelper.getTuple(), null,
valueResolver, valueResolver.getGlobalResolver());
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if (pkg != null) {
MVELDialectRuntimeData data = (MVELDialectRuntimeData)
pkg.getDialectRuntimeRegistry().getDialectData(this.id);
factory.setNextFactory(data.getFunctionFactory());
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEnabledExpression.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEnabledExpression.java
index b939f4e411..6d21651141 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEnabledExpression.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEnabledExpression.java
@@ -32,7 +32,6 @@ import org.drools.base.reteoo.BaseTuple;
import org.drools.base.reteoo.SortDeclarations;
import org.drools.base.rule.Declaration;
import org.drools.base.rule.accessor.Enabled;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.mvel2.integration.VariableResolverFactory;
@@ -91,7 +90,7 @@ public class MVELEnabledExpression
rule, null, tuple,
null, valueResolver, valueResolver.getGlobalResolver() );
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( this.id );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEvalExpression.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEvalExpression.java
index b82d5914d3..96e6d523f0 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEvalExpression.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELEvalExpression.java
@@ -29,7 +29,6 @@ import org.drools.base.definitions.rule.impl.RuleImpl;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
import org.drools.base.rule.accessor.EvalExpression;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.mvel2.integration.VariableResolverFactory;
@@ -94,7 +93,7 @@ public class MVELEvalExpression
factory );
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( this.id );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELObjectExpression.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELObjectExpression.java
index 58e28d345a..43fe25da81 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELObjectExpression.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELObjectExpression.java
@@ -29,7 +29,6 @@ import org.drools.base.definitions.rule.impl.RuleImpl;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
import org.drools.core.time.TimerExpression;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.mvel2.ParserConfiguration;
import org.mvel2.integration.VariableResolverFactory;
@@ -92,7 +91,7 @@ public class MVELObjectExpression implements MVELCompileable,
TimerExpression, E
null, null,
leftTuple, null, valueResolver, valueResolver.getGlobalResolver() );
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( this.id );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELReturnValueExpression.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELReturnValueExpression.java
index 4ad4e8ff93..22ada7230a 100644
---
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELReturnValueExpression.java
+++
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELReturnValueExpression.java
@@ -22,13 +22,10 @@ import java.io.Externalizable;
import java.io.IOException;
import java.io.ObjectInput;
import java.io.ObjectOutput;
-import java.util.Arrays;
import org.drools.base.base.ValueResolver;
import org.drools.base.definitions.InternalKnowledgePackage;
import org.drools.base.definitions.rule.impl.RuleImpl;
-import org.drools.core.impl.InternalRuleBase;
-import org.drools.core.impl.KnowledgeBaseImpl;
import org.drools.base.reteoo.BaseTuple;
import org.drools.base.rule.Declaration;
import org.drools.base.rule.accessor.FieldValue;
@@ -96,7 +93,7 @@ public class MVELReturnValueExpression
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalRuleBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( this.id );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELSalienceExpression.java
b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELSalienceExpression.java
index b9583df09d..4e391b6eec 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/expr/MVELSalienceExpression.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/expr/MVELSalienceExpression.java
@@ -33,7 +33,6 @@ import org.drools.base.rule.Declaration;
import org.drools.core.rule.consequence.InternalMatch;
import org.drools.base.rule.accessor.Salience;
import org.drools.base.time.TimeUtils;
-import org.drools.kiesession.rulebase.InternalKnowledgeBase;
import org.drools.mvel.MVELDialectRuntimeData;
import org.kie.api.definition.rule.Rule;
import org.kie.api.runtime.rule.Match;
@@ -97,7 +96,7 @@ public class MVELSalienceExpression
null,
valueResolver, valueResolver.getGlobalResolver() );
// do we have any functions for this namespace?
- InternalKnowledgePackage pkg =
((InternalKnowledgeBase)valueResolver.getRuleBase()).getPackage("MAIN");
+ InternalKnowledgePackage pkg =
valueResolver.getRuleBase().getPackage("MAIN");
if ( pkg != null ) {
MVELDialectRuntimeData data = ( MVELDialectRuntimeData )
pkg.getDialectRuntimeRegistry().getDialectData( this.id );
factory.setNextFactory( data.getFunctionFactory() );
diff --git
a/drools-mvel/src/main/java/org/drools/mvel/field/ClassFieldImpl.java
b/drools-mvel/src/main/java/org/drools/mvel/field/ClassFieldImpl.java
index 393bad3c55..15cf6a9250 100644
--- a/drools-mvel/src/main/java/org/drools/mvel/field/ClassFieldImpl.java
+++ b/drools-mvel/src/main/java/org/drools/mvel/field/ClassFieldImpl.java
@@ -19,7 +19,6 @@
package org.drools.mvel.field;
import org.drools.base.common.DroolsObjectInput;
-import org.drools.core.common.InternalWorkingMemory;
import org.drools.base.rule.accessor.FieldValue;
import java.io.Externalizable;
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]