Re: svn commit: r1587286 - in /jackrabbit/oak/trunk: oak-core/pom.xml oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/document/DocumentNodeStoreService.java oak-parent/pom.xml

2014-04-15 Thread Julian Reschke

On 2014-04-15 06:10, Chetan Mehrotra wrote:

Hi Julian,

On Tue, Apr 15, 2014 at 12:39 AM,  resc...@apache.org wrote:

-
Embed-Dependencycommons-dbcp,commons-pool,h2,json-simple/Embed-Dependency
+
Embed-Dependencycommons-dbcp,commons-pool,h2,json-simple,postgresql,db2,db2-license/Embed-Dependency
  Embed-Transitivetrue/Embed-Transitive


I believe this is a temporary change and would not be required for
final implementation? Would be helpful if we add a TODO/FIXME there
such that we remember to remove this later


OK.


Instead of embedding all such types of drivers/dbcp/pool etc within
oak-core it would be better to decouple them. For example one approach
can be

1. Have a bundle which embeds common-dbcp and required dependencies.
It would be responsible for registering a DataSource


Register a DataSource where?


2. Driver bundle would be fragments to the bundle #1 as host. With
JDBC 4.0 the Driver classes are provided as part of
META-INF/services/java.sql.Driver [1]. For such cases fragment bundles
can be avoided by having #1 monitor for such drivers and register them
programatically


Does this work in an OSGi context?


3. DocumentNodeStoreService should only have a reference to DataSource
and use that


How does it get the DataSource? Per JNDI?

Best regards, Julian



buildbot success in ASF Buildbot on oak-trunk-win7

2014-04-15 Thread buildbot
The Buildbot has detected a restored build on builder oak-trunk-win7 while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/oak-trunk-win7/builds/5315

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-win7

Build Reason: scheduler
Build Source Stamp: [branch jackrabbit/oak/trunk] 1587469
Blamelist: reschke

Build succeeded!

sincerely,
 -The Buildbot





buildbot failure in ASF Buildbot on oak-trunk-win7

2014-04-15 Thread buildbot
The Buildbot has detected a new failure on builder oak-trunk-win7 while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/oak-trunk-win7/builds/5317

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-win7

Build Reason: scheduler
Build Source Stamp: [branch jackrabbit/oak/trunk] 1587493
Blamelist: mduerig,reschke,thomasm

BUILD FAILED: failed compile

sincerely,
 -The Buildbot





buildbot success in ASF Buildbot on oak-trunk-win7

2014-04-15 Thread buildbot
The Buildbot has detected a restored build on builder oak-trunk-win7 while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/oak-trunk-win7/builds/5318

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-win7

Build Reason: scheduler
Build Source Stamp: [branch jackrabbit/oak/trunk] 1587504
Blamelist: chetanm

Build succeeded!

sincerely,
 -The Buildbot





buildbot failure in ASF Buildbot on oak-trunk-win7

2014-04-15 Thread buildbot
The Buildbot has detected a new failure on builder oak-trunk-win7 while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/oak-trunk-win7/builds/5321

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: bb-win7

Build Reason: scheduler
Build Source Stamp: [branch jackrabbit/oak/trunk] 1587580
Blamelist: mduerig,reschke

BUILD FAILED: failed compile

sincerely,
 -The Buildbot