[Building Sakai] OutOfMemory error when importing Gradebook Spreadsheet

[Aaron Zeckoski]

That makes sense to me Bryan. Can you please create a JIRA with your
patch (or just a description of the issue if you prefer)?
Thanks
-AZ


On Wed, Feb 3, 2010 at 1:42 AM, Bryan Bakotich <bakotibj at plu.edu> wrote:
> Hi all,
>
> We had a professor trying to upload a gradebook spreadsheet (via Gradebook
> -> Import Grades) crash our Sakai instance multiple times yesterday. It
> turns out that he had some text in the very last line (line 1048576) of the
> spreadsheet so there were over 1 million SpreadsheetRow objects created
> mainly for blank lines. For each blank row there was a line like this in
> catalina.out
>
> 2010-02-02 09:54:39,072 ERROR TP-Processor11
> org.sakaiproject.tool.gradebook.ui.SpreadsheetUploadBean - User  is unknown
> to this gradebook: java.lang.NullPointerException
>
> It doesn't seem like we should call the SpreadsheetRow constructor for blank
> rows. This can be fixed by adding a check in the csvToArray method in the
> SpreadSheetUploadBean.java file. I added this if statement in the while loop
> in the csvToArray method and it fixed the problem on our dev box:
>
> if (line.replaceAll(",", "").replaceAll("\"", "").equals("")) continue;
>
> Any thoughts or suggestions on a different place this should be caught.
> Should I create a JIRA for this? Thanks for the help.
>
> -Bryan
>
> --
> Bryan Bakotich
> Open Source Implementation Specialist
> Digital Media Center, Information & Technology Services
> Pacific Lutheran University
> Tacoma, WA 98447-0013
> Phone: 253-536-5021
>
> _______________________________________________
> sakai-dev mailing list
> sakai-dev at collab.sakaiproject.org
> http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
>
> TO UNSUBSCRIBE: send email to sakai-dev-unsubscribe at collab.sakaiproject.org
> with a subject of "unsubscribe"
>



-- 
Aaron Zeckoski (azeckoski (at) vt.edu)
Senior Research Engineer - CARET - University of Cambridge
https://twitter.com/azeckoski - http://www.linkedin.com/in/azeckoski
http://aaronz-sakai.blogspot.com/ - http://tinyurl.com/azprofile