Hangs when setting cell value to large JSON string

[tfmorris]

Original author: wbl...@myspace-inc.com (September 21, 2010 18:20:07)

What steps will reproduce the problem?

1. Edit any text cell
2. Paste the JSON below into the edit box (there's nothing special about it though, it will happen with any reasonably complex JSON).
3. Press Apply.

What is the expected output? What do you see instead?

Expect the value to be saved. Instead, it shows the "Working" lightbox/gif
which never goes away - you have to refresh the page. The edit does not succeed.

Seems like a JSON encoding problem.

What version of the product are you using? On what operating system?

Gridworks development build, Windows 7.

Please provide any additional information below.

{
"header": {
"code": "1",
"message": "Get Topic Successful"
},
"result": {
"freebase_id": "/en/anthony_rapp",
"topicuri": "anthony-rapp",
"daylife_id": "0fbnexm7mJboV",
"metadata": [
{
"name": "daylifeid",
"value": "0fbnexm7mJboV",
"source": "CrowdFusion"
},
{
"name": "myspaceid",
"value": "anthonyrapp",
"source": "CrowdFusion"
},
{
"name": "officialhomepage",
"value": "http://anthonyrapp.com/",
"source": "CrowdFusion"
},
{
"name": "topicname",
"value": "Anthony Rapp",
"source": "CrowdFusion"
},
{
"name": "freebaseimageid",
"value": [
"/m/02b8qt8",
"/wikipedia/images/commons_id/2985706"
],
"source": "Freebase"
},
{
"name": "freebasemachineid",
"value": "/m/03ysz4",
"source": "Freebase"
},
{
"name": "myspaceid",
"value": "anthonyrapp",
"source": "Freebase"
},
{
"name": "officialhomepage",
"value": "http://anthonyrapp.com/",
"source": "Freebase"
},
{
"name": "topicname",
"value": "Anthony Rapp",
"source": "Freebase"
},
{
"name": "wikipediaurlid",
"value": "1008352",
"source": "Freebase"
}
],
"images": [
{
"size": "129x150",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-150.jpg",
"source": "CrowdFusion"
},
{
"size": "140x140",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-140x140.jpg",
"source": "CrowdFusion"
},
{
"size": "150x113",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-150x113.jpg",
"source": "CrowdFusion"
},
{
"size": "150x150",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/10/anthony-rapp-1-150.jpg",
"source": "CrowdFusion"
},
{
"size": "150x75",
"url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-150x75.jpg",
"source": "CrowdFusion"
},
{
"size": "200x300",
"url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-200x300.jpg",
"source": "CrowdFusion"
},
{
"size": "211x328",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-211x328.jpg",
"source": "CrowdFusion"
},
{
"size": "300x211",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-328x211.jpg",
"source": "CrowdFusion"
},
{
"size": "300x240",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-520x240.jpg",
"source": "CrowdFusion"
},
{
"size": "300x322",
"url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-621x322.jpg",
"source": "CrowdFusion"
},
{
"size": "300x350",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-300x351.jpg",
"source": "CrowdFusion"
},
{
"size": "300x351x",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-300x351.jpg",
"source": "CrowdFusion"
},
{
"size": "55x55",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-55x55.jpg",
"source": "CrowdFusion"
},
{
"size": "621x322x",
"url": "http://crowdfusion.myspacecdn.com/media/Anthony Rapp-621x322.jpg",
"source": "CrowdFusion"
},
{
"size": "80x80",
"url": "http://crowdfusion.myspacecdn.com/media/2010/06/16/anthony-rapp-80x80.jpg",
"source": "CrowdFusion"
}
],
"topictype": [
{
"hub": "Celebrities"
}
],
"relatedtopics": [
{
"relatedtopicid": "/en/lucie_arnaz",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/mandy_patinkin",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/ben_affleck",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/bernadette_peters",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/peter_gallagher",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/adam_pascal",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/adam_pascal",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/adam_pascal",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/john_stamos",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/alan_cumming",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/alan_cumming",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/alan_cumming",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/anjelica_huston",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/elaine_stritch",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/elton_john",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/ethel_waters",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/hugh_jackman",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/idina_menzel",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/idina_menzel",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/idina_menzel",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/stephen_sondheim",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/joan_rivers",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/joel_grey",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/joey_lauren_adams",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/judith_light",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/liz_callaway",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/mark_wahlberg",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/matthew_broderick",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/michael_cerveris",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/parker_posey",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/philip_seymour_hoffman",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/suzanne_somers",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/taye_diggs",
"relation": "IsRelatedTo",
"source": "Daylife"
},
{
"relatedtopicid": "/en/tommy_lee",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/tommy_lee",
"relation": "IsRelatedTo",
"source": "CrowdFusion"
},
{
"relatedtopicid": "/en/tommy_lee",
"relation": "IsRelatedTo",
"source": "Daylife"
}
]
}
}

Original issue: http://code.google.com/p/google-refine/issues/detail?id=135

[tfmorris]

From wbl...@myspace-inc.com on September 21, 2010 18:30:35:
This was SVN revision 1261, last change date 2010-09-09.

[tfmorris]

From dfhu...@google.com on September 28, 2010 04:43:06:
Fixed by r1374.