> On Nov. 6, 2012, 1:07 p.m., Vicente J. Ruiz Jurado wrote: > > Tested and works correctly. > > > > Maybe you can also update the README (or some other doc) with info about > > this script. > > Vicente J. Ruiz Jurado wrote: > Sorry, I'm thinking that to have all the attachments in the same > directory can be a bad idea in terms of scalability. > > Anyway this patch it's necessary to have all previous attachments > coherent with: > https://reviews.apache.org/r/7471/ > but, what do you think?
I personally would prefer to keep the folders structure... But as you mentioned we need this script now. - Yuri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7832/#review13146 ----------------------------------------------------------- On Nov. 2, 2012, 12:56 p.m., Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7832/ > ----------------------------------------------------------- > > (Updated Nov. 2, 2012, 12:56 p.m.) > > > Review request for wave, Vicente J. Ruiz Jurado, Ali Lown, and Andrew > Kaplanov. > > > Description > ------- > > Adds ant script to migrate attachments to the newer version. The script just > copies the attachments to the root of the attachments folder to comply with > the new mechanism. > > > Diffs > ----- > > migrate-attachments.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/7832/diff/ > > > Testing > ------- > > Tested that the old attachments are displayed correctly in the up to date > version after running the migration script. > > > Thanks, > > Yuri Zelikov > >
