[xwiki-devs] [VOTE] Merge the 'feature-annotations-merged-with-comments' branch into 'master'
28 Feb
2012
28 Feb
'12
1:53 p.m.
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
29 Feb
29 Feb
7:54 a.m.
I commented on the pull request. I'm waiting for your replies before
casting my vote.
Thanks,
Marius
On Tue, Feb 28, 2012 at 2:53 PM, Eduard Moraru <enygma2002(a)gmail.com> wrote:
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
7:47 p.m.
Hi Marius,
I have updated the pull request based on your comments.
Thanks,
Eduard
On Wed, Feb 29, 2012 at 8:54 AM, Marius Dumitru Florea <
mariusdumitru.florea(a)xwiki.com> wrote:
I commented on the pull request. I'm waiting for
your replies before
casting my vote.
Thanks,
Marius
On Tue, Feb 28, 2012 at 2:53 PM, Eduard Moraru <enygma2002(a)gmail.com>
wrote:
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
2 Mar
2 Mar
12:19 p.m.
On Wed, Feb 29, 2012 at 8:47 PM, Eduard Moraru <enygma2002(a)gmail.com> wrote:
Hi Marius,
I have updated the pull request based on your comments.
+1 then :)
Thanks,
Marius
Thanks,
Eduard
On Wed, Feb 29, 2012 at 8:54 AM, Marius Dumitru Florea <
mariusdumitru.florea(a)xwiki.com> wrote:
I commented on the pull request. I'm waiting
for your replies before
casting my vote.
Thanks,
Marius
On Tue, Feb 28, 2012 at 2:53 PM, Eduard Moraru <enygma2002(a)gmail.com>
wrote:
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
1 Mar
1 Mar
4:01 p.m.
+1
On Tue, Feb 28, 2012 at 1:53 PM, Eduard Moraru <enygma2002(a)gmail.com> wrote:
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne
2 Mar
2 Mar
1:19 p.m.
On 02/28/2012 01:53 PM, Eduard Moraru wrote:
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
I commented too, otherwise looks fine to me.
I didn't read the migrator though, I'll leave that to the puller.
Thanks,
Anca
Here's my +1.
Thanks,
Eduard
----------
[1] http://markmail.org/thread/gecez5hdh5eci2le
[2] https://github.com/xwiki/xwiki-platform/pull/34/files
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
2:31 p.m.
On Fri, Mar 2, 2012 at 13:19, Anca Luca <lucaa(a)xwiki.com> wrote:
On 02/28/2012 01:53 PM, Eduard Moraru wrote:
I just have a look at the migrator and made some comments.
I have not followed closely the task, but I have the impression that the
migration could be done per document and not as a whole like you do which
execute the whole migration in a single transaction that could be really
heavy on the DB side.
This is not failsafe, and could cause many annoying issue if it goes wrong
due to the transaction size.
So, except if there is a good reason I have missed, the migration should at
least be splitted in smaller transactions, one per document seems easy. In
such implementation, restarting after failure would normally continue the
job where it have been left.
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
I commented too, otherwise looks fine to me.
I didn't read the migrator though, I'll leave that to the puller.
Thanks,
Anca
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
Here's my +1.
Thanks,
Eduard
----------
[1]
http://markmail.org/thread/**gecez5hdh5eci2le<http://markmail.org/thread…
[2]
https://github.com/xwiki/**xwiki-platform/pull/34/files<https://github.c…
______________________________**_________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/**mailman/listinfo/devs<http://lists.xwiki.org/ma…
______________________________**_________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/**mailman/listinfo/devs<http://lists.xwiki.org/ma…
21 Mar
21 Mar
11:43 a.m.
Hi Denis, all,
I have refactored the migration script to address the issue you pointed out.
Now, each document is migrated in a separate transaction, as you can see in
the hibernateMigrate() method [1].
Please have a look over the script [2] and see if there are any other
issues.
I`d like to merge this for 4.0M2
Thanks,
Eduard
----------
[1] https://github.com/xwiki/xwiki-platform/pull/34/files#L6R95
[2] https://github.com/xwiki/xwiki-platform/pull/34/files#diff-6
On Fri, Mar 2, 2012 at 3:31 PM, Denis Gervalle <dgl(a)softec.lu> wrote:
On Fri, Mar 2, 2012 at 13:19, Anca Luca
<lucaa(a)xwiki.com> wrote:
______________________________**_________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/**mailman/listinfo/devs<
http://lists.xwiki.org/mailman/listinfo/devs>
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
On 02/28/2012 01:53 PM, Eduard Moraru wrote:
I just have a look at the migrator and made some comments.
I have not followed closely the task, but I have the impression that the
migration could be done per document and not as a whole like you do which
execute the whole migration in a single transaction that could be really
heavy on the DB side.
This is not failsafe, and could cause many annoying issue if it goes wrong
due to the transaction size.
So, except if there is a good reason I have missed, the migration should at
least be splitted in smaller transactions, one per document seems easy. In
such implementation, restarting after failure would normally continue the
job where it have been left.
Hi devs,
As you know from the previous discussion [1], I`ve worked on integrating
the work done by Anca to merge Annotations with Comments.
I`ve fixed leftover integration problems and finished implementing the
migration script.
Please have a look at the pull request [2] and cast your vote whether we
can merge it into master for 4.0 or not.
I commented too, otherwise looks fine to me.
I didn't read the migrator though, I'll leave that to the puller.
Thanks,
Anca
> Here's my +1.
>
> Thanks,
> Eduard
>
> ----------
> [1] http://markmail.org/thread/**gecez5hdh5eci2le<
http://markmail.org/thread/gecez5hdh5eci2le>
https://github.com/xwiki/xwiki-platform/pull/34/files>
>
______________________________**_________________
> devs mailing list
> devs(a)xwiki.org
> http://lists.xwiki.org/**mailman/listinfo/devs<
http://lists.xwiki.org/mailman/listinfo/devs>
4615
days inactive
4637
days old
7 comments
5 participants
participants (5)
-
Anca Luca -
Denis Gervalle -
Eduard Moraru -
Marius Dumitru Florea -
Thomas Mortagne