Both the filter and filter element operators are completely broken in
the current Compose* implementation.  The filter operators were broken
by Dirk's filter module parameter patch. The code that would link the
right hand side of the operator doesn't do anything because at that time
no instances of FilterModule exist (FilterModule instances are created
from FilterModuleASTs in REXREF). As a result all filter operators won't
have a right hand side assigned. On top of that the implementation is
flawed. Because of filter module parameters every FilterModule instance
has unique filter instances, therefore it should also have unique filter
operator instances.

Filter element operators never worked because the linking algorithm was
flawed. The last filter element will point to itself through the filter
element operator. And this operator is also bugged with the unique
instance issue.

Both bugs look harmless, only FIRE2 references these operators and
completely ignores the right hand side value. It will assume the filters
and filter elements appear in order of execution. This assumption will
ofcourse render the operators partially useless. All other modules
should either use FIRE2's execution model or not rely on the function of
these operators (which is the case afaik).

In COPPER2 te operators will be properly linked. The unique instance
issue will not be fixed for the moment.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Composestar-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/composestar-developers

Reply via email to