|
[
Permlink
| « Hide
]
Stephen Marquard added a comment - 26-Sep-2007 02:05
Still an issue in trunk. This means SiteStats can't process this event, inter alia.
Sitestats error for reference:
WARN: Unable to parse contextId from event: section.add | (2008-01-22 20:45:18,996 org.sakaiproject.sitestats.impl.StatsUpdateManagerImpl_org.sakaiproject.sitestats.impl.StatsUpdateManagerImpl) java.lang.ArrayIndexOutOfBoundsException: 2 at org.sakaiproject.sitestats.impl.StatsUpdateManagerImpl.parseSiteId(StatsUpdateManagerImpl.java:744) at org.sakaiproject.sitestats.impl.StatsUpdateManagerImpl.preProcessEvent(StatsUpdateManagerImpl.java:355) at org.sakaiproject.sitestats.impl.StatsUpdateManagerImpl.run(StatsUpdateManagerImpl.java:303) at java.lang.Thread.run(Thread.java:595) Resolved in r46004 and tested on local trunk build.
Pls merge to 2-5-x. merged into 2-5-x with r46166
I sensed this issue has not been merged to 2.4.x. So I investigated a bit. It seems fix(patch) is pretty simple and easy though. I don't know how to reproduce the error in not-yet-patched sakai (e.g. http://nightly2.sakaiproject.org:8081/portal)
BTW, I attached the patch for 2.4.x Ok I have tested this by watching the sakai_event table before and after the code modification and can verify this fix:
before: 60 2008-06-19 00:03:59 section.add NULL 56014ab5-ec6d-409d-80f2-938be57a5288 m after: 99 2008-06-19 00:16:37 section.add /site/70789f0e-98c8-4504-003d-a2d4cf32c461/group/28a0feb3-ec85-48eb-0067-72288c39962b 05e00144-5e53-4f29-8091-ffbca80f0ef9 m I have also edited the initial description/title to make it a bit clearer what was going on. Have merged to 2.4.x branch: svn commit -m " Sending sections-impl/sakai/impl/src/java/org/sakaiproject/component/section/sakai/SectionManagerImpl.java Transmitting file data . Committed revision 47644. Will reassign to Nuno to check if this is going to mess up SiteStats. Steve, this is working perfectly and SiteStats is handling it well! (tested on updated local trunk instance).
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||