Re: [rules-users] Lots of org.drools.common.ScheduledAgendaItem instances in memory

2012-05-29 Thread Werner Stoop
Hi Wolfgang,

Thank you for your help. This sounds like a much better idea than what I
have at the moment.

I'll have to read up on queries in Drools first, though, because I've never
used them before.

On Tue, May 29, 2012 at 12:21 PM, Wolfgang Laun wrote:

> For this kind of clean-up (to get rid of events that have been around
> for 24h plus) you can insert a single event, let's call it EveryHour,
> and write a rule with a timer, to fire timer(int: 1h 1h). (If this is
> too coarse, use 15m 15 or whatever.) On the RHS, run a query to select
> all that you want to discard, and discard. The current time - 24h
> would have to be a parameter to the query.
>
> This should reduce the number of scheduled activations, at the cost of
> running the query; this depends on the number of Alarm events in the
> system.
>
> Other techniques I can think of might require some additional
> bookkeeping, so as to have all uncleared Alarms in some Collection.
> This could be tricky, depending on the number of state transitions,
> etc.
>
> -W
>
>
>
> On 29/05/2012, Werner Stoop  wrote:
> > Thanks Wolfgang,
> >
> > Yes, we do have a lot of events/hour, because it is a complex network
> we're
> > monitoring. Our system has been running for some time, but the Drools
> rules
> > engine is a new addition to attempt to manage some of the complexity.
> >
> > Perhaps I should clarify events and alarms: Our main system tracks alarms
> > within the network, but each alarm may have several events, like an event
> > when the alarm is first raised, an event when its status goes from major
> to
> > critical and an event when the alarm is cleared. So the main entity in
> our
> > rules is an Alarm, and whenever we get an event we insert a new Alarm
> into
> > the knowledge base if we've never seen the Alarm before, or update the
> > Alarm if we have.
> >
> > We have one other rule that removes all Alarms whose status haven't
> changed
> > for 24 hours, regardless of whether they have cleared or not. This rule's
> > syntax is very similar to the one from my previous email. We specifically
> > have this rule to try and keep the fact count in the rules engine
> > manageable.
> >
> >   rule "Old, Inactive Alarm?"
> >   timer(int: 30m 30m)
> >   salience -10
> >   when
> > $a : Alarm(severity != "cleared")
> >   then
> > double lastUpdate = minutesSince($a.getEventTime());
> > if(lastUpdate > 24 * 60) {
> > retract($a);
> > }
> >   end
> >
> > So what you said would explain the memory usage. All Alarms end up in
> "Old,
> > Inactive Alarm?"'s queue waiting for 24 hours.
> >
> > I'm going to disable this rule "Old, Inactive Alarm?" for the time being.
> > Unfortunately the nature of the problem means that I'll have to monitor
> it
> > for a day or two before I can draw any conclusions.
> >
> > It seems that the proper solution to this problem would be to get more
> > memory.
> >
> > Thank you,
> > Werner
> >
> > On Tue, May 29, 2012 at 9:35 AM, Wolfgang Laun
> > wrote:
> >
> >> On 29/05/2012, Werner Stoop  wrote:
> >> > Hi, thank you for your response.
> >> >
> >> > We use Drools 5.3.1 through Maven. When I invoke Drools, for each
> event
> >> > I
> >> > receive I do the following:
> >> >
> >> >   ksession.insert(obj);
> >> >   ksession.fireAllRules();
> >> >
> >> This is OK.
> >>
> >> >
> >> > Yes, we do use timers. In one case we want to remove alarms that have
> >> been
> >> > cleared for more than an hour from the knowledgebase. We don't remove
> >> them
> >> > immediately because some alarms clear briefly and then come back. The
> >> rule
> >> > I've written to handle this situation is the following:
> >> >
> >> > rule "Old Cleared Alarm?"
> >> > timer(int: 10m 10m)
> >> > salience -10
> >> > when
> >> > $a : Alarm(severity == "cleared")
> >> > then
> >> > double lastUpdate = minutesSince($a.getEventTime());
> >> > if(lastUpdate > 60) {
> >> > logger.debug("Alarm " + $a.getAlarmId() + " is old. Removing...");
> >> > retract($a);
> >> > }
> >> > end
> >> >
> >> > Is there any other way to write this? I've fo

Re: [rules-users] Lots of org.drools.common.ScheduledAgendaItem instances in memory

2012-05-29 Thread Werner Stoop
Thanks Wolfgang,

Yes, we do have a lot of events/hour, because it is a complex network we're
monitoring. Our system has been running for some time, but the Drools rules
engine is a new addition to attempt to manage some of the complexity.

Perhaps I should clarify events and alarms: Our main system tracks alarms
within the network, but each alarm may have several events, like an event
when the alarm is first raised, an event when its status goes from major to
critical and an event when the alarm is cleared. So the main entity in our
rules is an Alarm, and whenever we get an event we insert a new Alarm into
the knowledge base if we've never seen the Alarm before, or update the
Alarm if we have.

We have one other rule that removes all Alarms whose status haven't changed
for 24 hours, regardless of whether they have cleared or not. This rule's
syntax is very similar to the one from my previous email. We specifically
have this rule to try and keep the fact count in the rules engine
manageable.

  rule "Old, Inactive Alarm?"
  timer(int: 30m 30m)
  salience -10
  when
$a : Alarm(severity != "cleared")
  then
double lastUpdate = minutesSince($a.getEventTime());
if(lastUpdate > 24 * 60) {
retract($a);
}
  end

So what you said would explain the memory usage. All Alarms end up in "Old,
Inactive Alarm?"'s queue waiting for 24 hours.

I'm going to disable this rule "Old, Inactive Alarm?" for the time being.
Unfortunately the nature of the problem means that I'll have to monitor it
for a day or two before I can draw any conclusions.

It seems that the proper solution to this problem would be to get more
memory.

Thank you,
Werner

On Tue, May 29, 2012 at 9:35 AM, Wolfgang Laun wrote:

> On 29/05/2012, Werner Stoop  wrote:
> > Hi, thank you for your response.
> >
> > We use Drools 5.3.1 through Maven. When I invoke Drools, for each event I
> > receive I do the following:
> >
> >   ksession.insert(obj);
> >   ksession.fireAllRules();
> >
> This is OK.
>
> >
> > Yes, we do use timers. In one case we want to remove alarms that have
> been
> > cleared for more than an hour from the knowledgebase. We don't remove
> them
> > immediately because some alarms clear briefly and then come back. The
> rule
> > I've written to handle this situation is the following:
> >
> > rule "Old Cleared Alarm?"
> > timer(int: 10m 10m)
> > salience -10
> > when
> > $a : Alarm(severity == "cleared")
> > then
> > double lastUpdate = minutesSince($a.getEventTime());
> > if(lastUpdate > 60) {
> > logger.debug("Alarm " + $a.getAlarmId() + " is old. Removing...");
> > retract($a);
> > }
> > end
> >
> > Is there any other way to write this? I've found that I can't put the
> > minutesSince($a.getEventTime()) in the rule's when-clause.
>
> It's fine as you have it; it would not be evaluated correctly on the LHS.
>
> But considering 200 events, if they were all Alarm, you'd have a
> rate of 17800 events/hour, and so you'd have that many scheduled
> agenda items.
>
> What about the other timer rules for other Event types? Are there
> similar scenarios?
>
> -W
>
> >
> > Thank you,
> > Werner
> >
> > On Tue, May 29, 2012 at 8:10 AM, Wolfgang Laun
> > wrote:
> >
> >> Just to make sure: How do you invoke the Engine? (I suppose you don't
> >> call with a limit for rule firings.)
> >>
> >> Unless it's a bug (BTW: your Drools version is?), it's due to one or
> >> more of your rules.
> >>
> >> Are you using timers? How?
> >>
> >> A detailed investigation of the whereabouts of these
> >> ScheduledAgendaItem objects might be done by investigating (via the
> >> unstable API) the Agenda and its various components.
> >>
> >> -W
> >>
> >> On 28/05/2012, Werner Stoop  wrote:
> >> > Hi,
> >> >
> >> > We're using Drools with a StatefulKnowledgeSession to process events
> >> coming
> >> > from equipment in our network. The system draws conclusions about the
> >> state
> >> > of the equipment and writes those conclusions to a table in our
> >> > database. All our rules work as we expected and the system produces
> the
> >> > correct results.
> >> >
> >> > However, the memory usage of the JVM steadily goes up when the system
> >> runs
> >> > for extended periods of time until we start getting
> >> > OutOfMemoryExceptions
> 

Re: [rules-users] Lots of org.drools.common.ScheduledAgendaItem instances in memory

2012-05-28 Thread Werner Stoop
Hi, thank you for your response.

We use Drools 5.3.1 through Maven. When I invoke Drools, for each event I
receive I do the following:

  ksession.insert(obj);
  ksession.fireAllRules();

where ksession was previously created through

  kbase = KnowledgeBaseFactory.newKnowledgeBase();
  kbase.addKnowledgePackages(kbuilder.getKnowledgePackages());
  ksession = kbase.newStatefulKnowledgeSession();

Yes, we do use timers. In one case we want to remove alarms that have been
cleared for more than an hour from the knowledgebase. We don't remove them
immediately because some alarms clear briefly and then come back. The rule
I've written to handle this situation is the following:

rule "Old Cleared Alarm?"
timer(int: 10m 10m)
salience -10
when
$a : Alarm(severity == "cleared")
then
double lastUpdate = minutesSince($a.getEventTime());
if(lastUpdate > 60) {
logger.debug("Alarm " + $a.getAlarmId() + " is old. Removing...");
retract($a);
}
end

Is there any other way to write this? I've found that I can't put the
minutesSince($a.getEventTime()) in the rule's when-clause.

Thank you,
Werner

On Tue, May 29, 2012 at 8:10 AM, Wolfgang Laun wrote:

> Just to make sure: How do you invoke the Engine? (I suppose you don't
> call with a limit for rule firings.)
>
> Unless it's a bug (BTW: your Drools version is?), it's due to one or
> more of your rules.
>
> Are you using timers? How?
>
> A detailed investigation of the whereabouts of these
> ScheduledAgendaItem objects might be done by investigating (via the
> unstable API) the Agenda and its various components.
>
> -W
>
> On 28/05/2012, Werner Stoop  wrote:
> > Hi,
> >
> > We're using Drools with a StatefulKnowledgeSession to process events
> coming
> > from equipment in our network. The system draws conclusions about the
> state
> > of the equipment and writes those conclusions to a table in our
> > database. All our rules work as we expected and the system produces the
> > correct results.
> >
> > However, the memory usage of the JVM steadily goes up when the system
> runs
> > for extended periods of time until we start getting OutOfMemoryExceptions
> > and the server has to be restarted. This is in spite of the fact that the
> > fact count reported by
> > the StatefulKnowledgeSession.getFactCount() stays reasonably stable,
> > with around 30 000 facts (give or take) at any point in time.
> >
> > I have run the Eclipse Memory Analyzer tool (http://www.eclipse.org/mat/
> )
> > against heap dumps from the JVM several times now, and every time it
> > reports more and more instances
> > of org.drools.common.ScheduledAgendaItem referenced from one instance of
> > java.lang.Object[]
> >
> > To be concrete, since this morning the uptime is more than 112 hours in
> > total, during which the system has processed little over 2 000 000 events
> > from the network. It has 29 000 facts in the knowledge session, yet in
> the
> > heap dump we see 829 632 instances of
> > org.drools.common.ScheduledAgendaItem.
> >
> > What is the ScheduledAgendaItem for? Is there something wrong with my
> rules
> > that causes this many instances to be held? Is there something I should
> do
> > to release these instances or the Object[] holding on to them?
> >
> > Thanks,
> > Werner Stoop
> >
> ___
> rules-users mailing list
> rules-users@lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/rules-users
>
___
rules-users mailing list
rules-users@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-users


[rules-users] Lots of org.drools.common.ScheduledAgendaItem instances in memory

2012-05-28 Thread Werner Stoop
Hi,

We're using Drools with a StatefulKnowledgeSession to process events coming
from equipment in our network. The system draws conclusions about the state
of the equipment and writes those conclusions to a table in our
database. All our rules work as we expected and the system produces the
correct results.

However, the memory usage of the JVM steadily goes up when the system runs
for extended periods of time until we start getting OutOfMemoryExceptions
and the server has to be restarted. This is in spite of the fact that the
fact count reported by
the StatefulKnowledgeSession.getFactCount() stays reasonably stable,
with around 30 000 facts (give or take) at any point in time.

I have run the Eclipse Memory Analyzer tool (http://www.eclipse.org/mat/)
against heap dumps from the JVM several times now, and every time it
reports more and more instances
of org.drools.common.ScheduledAgendaItem referenced from one instance of
java.lang.Object[]

To be concrete, since this morning the uptime is more than 112 hours in
total, during which the system has processed little over 2 000 000 events
from the network. It has 29 000 facts in the knowledge session, yet in the
heap dump we see 829 632 instances of org.drools.common.ScheduledAgendaItem.

What is the ScheduledAgendaItem for? Is there something wrong with my rules
that causes this many instances to be held? Is there something I should do
to release these instances or the Object[] holding on to them?

Thanks,
Werner Stoop
___
rules-users mailing list
rules-users@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-users