ddavisqa / google-refine

Automatically exported from code.google.com/p/google-refine
0 stars 0 forks source link

Fusion Tables CSV import produces NullPointerException #334

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Per 
http://code.google.com/apis/fusiontables/groups/fusionTablesApiGroup.html?place=
topic%2Ffusion-tables-users-group%2FbWAcZnuOZlg%2Fdiscussion

Attempting to import a Google Fusion Tables CSV file produced by the following 
query

  http://www.google.com/fusiontables/api/query?sql=SELECT * FROM 274408 

causes a NullPointerException in the GData importer.  Since this isn't a GData 
query, it shouldn't be processed by the GData importer and should be allowed to 
fall through to the CSV importer.

Original issue reported on code.google.com by tfmorris on 14 Feb 2011 at 8:53

GoogleCodeExporter commented 9 years ago
I've tightened up the URL pattern matching so the Google Spreadsheets/Fusion 
Table importer doesn't get invoked, but that URL fails in a different way now 
in the generic importer getting an HTTP 400 Bad Request error even though the 
same URL (http://www.google.com/fusiontables/api/query?sql=SELECT * FROM 
274408) works in the browser.  A quoting issue perhaps?  If someone else wants 
to take a look, feel free, otherwise I'll try to circle back at some point in 
the future.

Thread [pool-1-thread-3] (Suspended (exception IOException))    
    HttpURLConnection.getInputStream() line: 1313 [local variables unavailable] 
    CreateProjectCommand.internalImportURL(HttpServletRequest, Project, ProjectMetadata, Properties, String) line: 415  
    CreateProjectCommand.internalImport(HttpServletRequest, Project, ProjectMetadata, Properties) line: 187 

Original comment by tfmorris on 14 Feb 2011 at 10:31

GoogleCodeExporter commented 9 years ago
Fixed in CreateProjectCommand by making sure URLs are encoded before using them.

Original comment by tfmorris on 15 Feb 2011 at 11:15

GoogleCodeExporter commented 9 years ago

Original comment by tfmorris on 9 Jun 2011 at 7:58