Closed (fixed)
Project:
Feed Element Mapper
Version:
6.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jan 2009 at 16:54 UTC
Updated:
13 Apr 2009 at 12:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
alex_b commentedCould you post an (anonymized) version of the CSV file you're processing?
Comment #2
techninja commentedI have the same problem, and so far as I can tell it dies after trying to handle the 64th column.. everything including #63 work perfectly, then blam! 64 breaks it.
This is while trying to map the csv columns to CCK fields through feedapi mapper (/node/[feed-nid]/map)
Says it's out of memory (no matter how much you give it), and the dblog spits out
Not quite sure what's going on... in the mean time, I'll be putting a workaround that truncates incoming data to those first 63 cols (as I don't need any after that in my current case) though I'd imagine others wouldn't be so fortunate.
The attached file is just two rows and header for 64 cols. Remove #64 and you can suddenly map them.
Comment #3
techninja commentedWell, much as I've proven this to be the case, it looks like it's not the CSV parser. I converted my 70+column CSV to XML, and ran it through the SimplePIE parser off into feedapi and it died on TWO completely different servers. Eating up as much memory as it could and then crapping out trying to allocate some incredibly tiney amount (~ 30 to 70 bytes).
After some careful pruning and trimming, I was able to get my xml to FINALLY get to the map page by, you guessed it, cutting the number of flat individual xml nodes to less than 64.
So.. I'd just like our faithful csv_parser maintainer to see this off in it's new life, to be passed to the feedapi issue queue (assuming there's not already a dupe issue there waiting...)
Comment #4
aron novakComment #5
techninja commented[Changing title to be more helpful and specific]
Comment #6
e2thex commentedI was having the same problem, when using a RSS parser (i wrote that parse all element in the feed items). The patch fixed my problem.
Comment #7
aron novake2thex: thanks for the review, this means reviewed and tested by the community :)
Comment #8
aron novak