Page Comparison

[11:22:07 CDT(-0500)] <lfuller1> Got pulled in to look over a problem where a university using uportal is running out of memory. I showed them how to take a heap dump, which they did the next time they ran into trouble. At present it seems to take them roughly 24 hours to get in a bad situation.
[11:22:15 CDT(-0500)] <lfuller1> Looking at the heap dump they saw this:
[11:22:18 CDT(-0500)] <lfuller1> One instance of "org.jasig.portal.events.handlers.QueueingEventHandler" loaded by "org.apache.catalina.loader.WebappClassLoader @ 0x2aaab3b25ba0" occupies 1,814,492,936 (77.51%) bytes.
[11:22:43 CDT(-0500)] <lfuller1> Which... looks as if it could be related to the stats. Anyone run into a similar issue?
[11:22:43 CDT(-0500)] <EricDalquist> is this the same issue that was talked about on the user list?
[11:22:59 CDT(-0500)] <lfuller1> no idea. Will go look at the user list.
[11:23:02 CDT(-0500)] <EricDalquist> where it sounded like Quartz magically stopped calling the stats flush list
[11:23:06 CDT(-0500)] <EricDalquist> er flush trigger
[11:23:14 CDT(-0500)] <lfuller1> lame.
[11:23:25 CDT(-0500)] <lfuller1> this has been happening pretty consistently for these guys
[11:23:59 CDT(-0500)] <EricDalquist> yeah
[11:24:11 CDT(-0500)] <EricDalquist> I'm trying to remember who it was happening too
[11:24:12 CDT(-0500)] <lfuller1> They just discovered the problem this morning though.
[11:24:14 CDT(-0500)] <EricDalquist> was it parker grimes?
[11:24:50 CDT(-0500)] <EricDalquist> nope ... Paul Gazda
[11:25:06 CDT(-0500)] <EricDalquist> hrm
[11:25:09 CDT(-0500)] <EricDalquist> maybe not that either
[11:25:26 CDT(-0500)] <lfuller1> did they end up resolving the issue or just punt and comment out the stats?
[11:25:56 CDT(-0500)] <EricDalquist> ah it was Curtis
[11:25:59 CDT(-0500)] <EricDalquist> finally found the thread
[11:26:43 CDT(-0500)] <EricDalquist> http://jasig.275507.n4.nabble.com/uportal-statistics-td2262749.html#a2262749
[11:27:07 CDT(-0500)] <EricDalquist> I'm not sure I ever got a really good answer about what debugging they added
[11:27:40 CDT(-0500)] <EricDalquist> my thought was to turn quartz logging to debug
[11:27:49 CDT(-0500)] <EricDalquist> and add logging at the entry point to the stats flush call
[11:27:49 CDT(-0500)] <lfuller1> yeah, once you said the name I found the thread..
[11:29:19 CDT(-0500)] <EricDalquist> and see if it could be figured out why the flush call wasn't triggering
[15:22:56 CDT(-0500)] <EricDalquist> lfuller1: what ever happened with that portlet entity cleanup patch?
[15:23:03 CDT(-0500)] <EricDalquist> I can't remember where we left that discussion
[15:30:47 CDT(-0500)] <lfuller1> Thought we stopped mid discussion. Last I heard I needed to change from a string to an Id object.
[15:30:53 CDT(-0500)] <lfuller1> that is the only thing that had been confirmed
[15:30:58 CDT(-0500)] <EricDalquist> oh yeah, ok
[15:31:09 CDT(-0500)] <EricDalquist> we're in the midst of our upgrade to 3.2 right now
[15:31:16 CDT(-0500)] <EricDalquist> so I think I'll be applying it locally pretty soon
[15:31:28 CDT(-0500)] <lfuller1> was not sure if I should upload a patch for the patch and trunk or if I should commit them and then use crucible to finish the code review.
[15:32:06 CDT(-0500)] <lfuller1> willing to do whatever you want, will have time early tomorrow morning to take care of it.
[15:32:39 CDT(-0500)] <lfuller1> sorry I meant patches branches and trunk. Patches for those.
[15:32:55 CDT(-0500)] <EricDalquist> if you want to make that ID change first then feel free to commit it to the various places it is needed
[15:33:00 CDT(-0500)] <EricDalquist> we can do one more review after that
[15:33:07 CDT(-0500)] <EricDalquist> but I don't think anything else will come up
[15:33:07 CDT(-0500)] <lfuller1> ok, will do
[15:33:21 CDT(-0500)] <EricDalquist> the only other thing I can see being useful for people is coming up with a SQL query to clean up an existing DB
[15:33:24 CDT(-0500)] <lfuller1> hope not
[15:33:42 CDT(-0500)] <lfuller1> will post that to the jira ticket.
[15:34:01 CDT(-0500)] <EricDalquist> oh neat ... you already have that?
[15:36:26 CDT(-0500)] <lfuller1> the update?
[15:36:28 CDT(-0500)] <lfuller1> yeah
[15:36:46 CDT(-0500)] <lfuller1> think I stuck it in tomboy somewhere... will dig it up tomorrow