[wp-trac] Re: [WordPress Trac] #3178: Import should not fail if file size larger than max php memory

[WordPress Trac]

#3178: Import should not fail if file size larger than max php memory
---------------------------------------------+------------------------------
 Reporter:  technosailor                     |        Owner:  anonymous
     Type:  defect                           |       Status:  reopened 
 Priority:  normal                           |    Milestone:  2.2      
Component:  Administration                   |      Version:  2.0.4    
 Severity:  normal                           |   Resolution:           
 Keywords:  import MT filesize memory limit  |  
---------------------------------------------+------------------------------
Changes (by foolswisdom):

  * keywords:  import MT => import MT filesize memory limit
  * summary:  MT Importer Breaks if File Size takes too much memory =>
              Import should not fail if file size larger than
              max php memory
  * milestone:  2.1 => 2.2

Old description:

> I imagine my patch is ill-advisable so open to other recommendations.
> This worked on a client platform where the memory_limit was set in
> php.ini to be 12M, far smaller than was needed to do an MT import. This
> patch temporarily adjusts the memory_limit to 50M which I'd think would
> be big enough for just about anyone.
>
> A better solution might be to check file size against memory allocation
> required and provide a more user-friendly error or prompt to split the MT
> file into two.

New description:

 MT Importer Breaks if File Size takes too much memory

 Proposed solution: [[br]]
 Read X MB of the file finding a convenient break point, import and
 continue where left off.

Comment:

 r4388 is relevant

 Changing milestone to 2.2

 Any implementations from other projects that can be copied/borrowed?

 Updated Description from:[[br]]
 I imagine my patch is ill-advisable so open to other recommendations. This
 worked on a client platform where the memory_limit was set in php.ini to
 be 12M, far smaller than was needed to do an MT import. This patch
 temporarily adjusts the memory_limit to 50M which I'd think would be big
 enough for just about anyone.

 A better solution might be to check file size against memory allocation
 required and provide a more user-friendly error or prompt to split the MT
 file into two.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/3178#comment:8>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software