https://bugzilla.wikimedia.org/show_bug.cgi?id=29250

--- Comment #2 from Michael Dale <d...@ucsc.edu> 2011-11-17 23:16:20 UTC ---
Created attachment 9488
  --> https://bugzilla.wikimedia.org/attachment.cgi?id=9488
Resumable upload using database to sync state patch

I think some things can still be cleaned up, but all working, so I wanted to
get feedback.  

Changes Overview: 

* Subclass out UploadFromChunks having it extend UploadFromFile since it was
getting kind of messy having UploadFromFile handling both cases.

* The main execute function in ApiUpload was getting a little long and hard to
follow, re factored at least the result context part into a few private
methods.

* Added sa_chunk_inx filed to uploadstash for keeping track of the chunk inx. 
Added to tables.sql and mysqlUpdater.php / patch-uploadstash_chunk.sql )

* Deprecated appendToUploadFile and appendFinish in UploadBase for 1.19 since
we won't have append around in the future.

* removed getFileTempname from UploadFromFile ( not used anywhere )

* To simplify implementation mFile is left as a reference to the original first
stashed chunk, and just put chunks into the stashed folder by number

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to