You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2012/02/02 13:44:01 UTC
svn commit: r1239586 - in /directory/apacheds/trunk: all/pom.xml
jdbm-partition/pom.xml protocol-ldap/pom.xml
Author: elecharny
Date: Thu Feb 2 12:44:01 2012
New Revision: 1239586
URL: http://svn.apache.org/viewvc?rev=1239586&view=rev
Log:
Point to the modified version of JDBM.
Modified:
directory/apacheds/trunk/all/pom.xml
directory/apacheds/trunk/jdbm-partition/pom.xml
directory/apacheds/trunk/protocol-ldap/pom.xml
Modified: directory/apacheds/trunk/all/pom.xml
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/all/pom.xml?rev=1239586&r1=1239585&r2=1239586&view=diff
==============================================================================
--- directory/apacheds/trunk/all/pom.xml (original)
+++ directory/apacheds/trunk/all/pom.xml Thu Feb 2 12:44:01 2012
@@ -73,7 +73,7 @@
<dependency>
<groupId>${project.groupId}</groupId>
- <artifactId>apacheds-jdbm</artifactId>
+ <artifactId>apacheds-jdbm2</artifactId>
</dependency>
<dependency>
Modified: directory/apacheds/trunk/jdbm-partition/pom.xml
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/jdbm-partition/pom.xml?rev=1239586&r1=1239585&r2=1239586&view=diff
==============================================================================
--- directory/apacheds/trunk/jdbm-partition/pom.xml (original)
+++ directory/apacheds/trunk/jdbm-partition/pom.xml Thu Feb 2 12:44:01 2012
@@ -57,7 +57,7 @@
<dependency>
<groupId>${project.groupId}</groupId>
- <artifactId>apacheds-jdbm</artifactId>
+ <artifactId>apacheds-jdbm2</artifactId>
</dependency>
<dependency>
Modified: directory/apacheds/trunk/protocol-ldap/pom.xml
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/protocol-ldap/pom.xml?rev=1239586&r1=1239585&r2=1239586&view=diff
==============================================================================
--- directory/apacheds/trunk/protocol-ldap/pom.xml (original)
+++ directory/apacheds/trunk/protocol-ldap/pom.xml Thu Feb 2 12:44:01 2012
@@ -60,7 +60,7 @@
<dependency>
<groupId>${project.groupId}</groupId>
- <artifactId>apacheds-jdbm</artifactId>
+ <artifactId>apacheds-jdbm2</artifactId>
</dependency>
<dependency>
Re: svn commit: r1239586 - in /directory/apacheds/trunk: all/pom.xml
jdbm-partition/pom.xml protocol-ldap/pom.xml
Posted by Alex Karasulu <ak...@apache.org>.
On Sat, Feb 4, 2012 at 2:44 AM, Emmanuel Lecharny <el...@gmail.com>wrote:
> On 2/3/12 10:51 PM, Alex Karasulu wrote:
>
>> This is a really bold move here Emmanuel. The txn branch is not even alpha
>> and a serious change that will effect the server. I thought this was
>> something we would slowly start to transition into the main branch of
>> development.
>>
>> I don't know if it should require a vote but maybe we should talk about
>> this a little bit no?
>>
>> Point to the modified version of JDBM.
>>
>
> Sure. let me explain why I did that move, and why it's not critical.
>
> Having a MVCC backend could allow us to solve the problem we have with
> concurrent modifications and searches. We don't necessarily need to have
> the full in-memory MVCC Selcuk is working on in its branch in order to
> benefit from part of what he already have done : if we protect the
> modifications in the jdbm-partition against concurrent access to the
> backend, then searches and modifications could probably safely be executed
> concurrently.
>
> I need to test this part, and I don't want to do that in a branch, because
> it's too much a pain to merge it back while we are fixing many other issues
> in the server.
>
> Hopefully, this move is just impacting three poms and reverting back to
> jdbm is just a matter to point back to the previous version : just a breeze.
>
> I should have told the list about this change before doing it, my bad.
> Sadly, I made a mistake and had to commit the modifications in the poms
> because I broke the trunk this morning with a partial commit. This is why
> we now point to jdbm2. This can easily be fixed, and we can safely revert
> to jdbm.
>
> --
> Regards,
> Cordialement,
> Emmanuel Lécharny
> www.iktek.com
>
>
Thanks for clarifying and giving a thorough explanation. I was just
surprised to see the move and wanted your thoughts.
--
Best Regards,
-- Alex
Re: svn commit: r1239586 - in /directory/apacheds/trunk: all/pom.xml
jdbm-partition/pom.xml protocol-ldap/pom.xml
Posted by Emmanuel Lecharny <el...@gmail.com>.
On 2/3/12 10:51 PM, Alex Karasulu wrote:
> This is a really bold move here Emmanuel. The txn branch is not even alpha
> and a serious change that will effect the server. I thought this was
> something we would slowly start to transition into the main branch of
> development.
>
> I don't know if it should require a vote but maybe we should talk about
> this a little bit no?
>
> Point to the modified version of JDBM.
Sure. let me explain why I did that move, and why it's not critical.
Having a MVCC backend could allow us to solve the problem we have with
concurrent modifications and searches. We don't necessarily need to have
the full in-memory MVCC Selcuk is working on in its branch in order to
benefit from part of what he already have done : if we protect the
modifications in the jdbm-partition against concurrent access to the
backend, then searches and modifications could probably safely be
executed concurrently.
I need to test this part, and I don't want to do that in a branch,
because it's too much a pain to merge it back while we are fixing many
other issues in the server.
Hopefully, this move is just impacting three poms and reverting back to
jdbm is just a matter to point back to the previous version : just a breeze.
I should have told the list about this change before doing it, my bad.
Sadly, I made a mistake and had to commit the modifications in the poms
because I broke the trunk this morning with a partial commit. This is
why we now point to jdbm2. This can easily be fixed, and we can safely
revert to jdbm.
--
Regards,
Cordialement,
Emmanuel Lécharny
www.iktek.com
Re: svn commit: r1239586 - in /directory/apacheds/trunk: all/pom.xml
jdbm-partition/pom.xml protocol-ldap/pom.xml
Posted by Alex Karasulu <ak...@apache.org>.
This is a really bold move here Emmanuel. The txn branch is not even alpha
and a serious change that will effect the server. I thought this was
something we would slowly start to transition into the main branch of
development.
I don't know if it should require a vote but maybe we should talk about
this a little bit no?
Point to the modified version of JDBM.
>
> Modified: directory/apacheds/trunk/all/pom.xml
> URL:
> http://svn.apache.org/viewvc/directory/apacheds/trunk/all/pom.xml?rev=1239586&r1=1239585&r2=1239586&view=diff
>
> ==============================================================================
> --- directory/apacheds/trunk/all/pom.xml (original)
> +++ directory/apacheds/trunk/all/pom.xml Thu Feb 2 12:44:01 2012
> @@ -73,7 +73,7 @@
>
> <dependency>
> <groupId>${project.groupId}</groupId>
> - <artifactId>apacheds-jdbm</artifactId>
> + <artifactId>apacheds-jdbm2</artifactId>
> </dependency>
>
>
--
Best Regards,
-- Alex