Merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba

[Cezar Andrei]

Add new oracle-nosql-db module.

This has been renamed and it also contains multi-get and multi-delete functions.

Add new oracle-nosql-db module.

[Cezar Andrei (cezar-andrei)]

Please review the code in https://code.launchpad.net/~zorba-coders/zorba/f-nosqldb-module for the new nosql-db module.

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2012-11-08T22-48-55.136Z/log.html

[Matthias Brantner (matthias-brantner)]

I don't like the name nosqldb-module because it's confusing/ambiguous. Would be nice if the name Oracle appeared there.

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2012-11-08T22-48-55.136Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Pending.

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2012-11-26T16-43-57.589Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2012-11-26T16-43-57.589Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Pending.

[Juan Zacarias (juan457)]

review: Approve

[Chris Hillery (ceejatec)]

The oracle-nosql-db module test cases have no expected results, so they don't pass.

review: Needs Fixing

[Chris Hillery (ceejatec)]

Also, should add a mention of the new module to the ChangeLog.

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2012-12-19T02-08-46.766Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2012-12-19T02-08-46.766Z is finished.
  The final status was:

  No tests were run - build or configure step must have failed.

  Not commiting changes.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2012-12-19T02-36-50.72Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2012-12-19T02-36-50.72Z is finished.
  The final status was:

  2 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Chris Hillery (ceejatec)]

I've fixed one XQDoc problem (un-terminated <pre> elements), but now there are a number of documentation issues to clean up to please the RQ. Run "ctest -R moduleDocumentation -V" to see them.

665: processing module: http://www.zorba-xquery.com/modules/oracle-nosqldb
665: ERROR: Missing module description
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: ERROR: Missing module author name
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: ERROR: Missing explicit XQuery version declaration in prolog
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:remove
665: Arity: 2
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:put-text
665: Arity: 3
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:put-text
665: Arity: 4
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:put-binary
665: Arity: 3
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:put-binary
665: Arity: 4
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:flush
665: Arity: 1
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:touch
665: Arity: 3
665: ERROR: Return value not documented;
665: Module: http://www.zorba-xquery.com/modules/couchbase
665: Function: cb:disconnect
665: Arity: 1

review: Needs Fixing

[Chris Hillery (ceejatec)]

Comment from Matthias:

- rename delete-value function to remove to be consistent with Couchbase
- document errors that may be raised by the functions
- can we make the get and put functions take sequences of keys and values, respectively
(e.g. nosql:get-text($db as xs:anyURI, $keys as object()* ) as object()*)

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2012-12-23T09-35-47.636Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2012-12-23T09-35-47.636Z is finished.
  The final status was:

  1 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Chris Hillery (ceejatec)]

According to Cezar:

> Comment from Matthias:
>
> - rename delete-value function to remove to be consistent with Couchbase
> - document errors that may be raised by the functions

Both done.

> - can we make the get and put functions take sequences of keys and values,
> respectively
> (e.g. nosql:get-text($db as xs:anyURI, $keys as object()* ) as object()*)

There are 3 kinds of functions in this module:
- convenience functions for working with strings as keys, just like a java Map.
- raw that gives access to major and minor key components
- and advanced called multi-* that gives access to sub-ranges of keys.

I rather not add the $keys as object()* to avoid confusion with the multi-* functions that use sub-ranges.

[Zorba Build Bot (zorba-buildbot)]

Attempt to merge into lp:zorba failed due to conflicts:

text conflict in ChangeLog

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-04T08-01-50.372Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2013-01-04T08-01-50.372Z is finished.
  The final status was:

  5 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Zorba Build Bot (zorba-buildbot)]

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

[Chris Hillery (ceejatec)]

review: Approve

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-04T16-13-43.369Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2013-01-04T16-13-43.369Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Approve, 1 Pending.

[Matthias Brantner (matthias-brantner)]

I like the module a lot. There are only a couple of minor things that need fixing:

- There are a bunch of typos in the documentation. Could you please run it through a spell-checker?

For example,

s/corespond/correspond
s/initiaized/initialized
s/corectly/correctly

- the jsoniq functions module doesn't need to be included
- variable names and errors should not use camel-case but dashes, e.g. subRange => sub-range

review: Needs Fixing

[Cezar Andrei (cezar-andrei)]

> I like the module a lot. There are only a couple of minor things that need
> fixing:
>
> - There are a bunch of typos in the documentation. Could you please run it
> through a spell-checker?
>
> For example,
>
> s/corespond/correspond
> s/initiaized/initialized
> s/corectly/correctly
Fixed.

>
> - the jsoniq functions module doesn't need to be included
Yes it does, it's used at line 67: jn:members($helper-host-ports)

> - variable names and errors should not use camel-case but dashes, e.g.
> subRange => sub-range
Done.

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T05-11-01.06Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2013-01-09T05-11-01.06Z is finished.
  The final status was:

  1 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T05-53-57.216Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2013-01-09T05-53-57.216Z is finished.
  The final status was:

  1 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T06-53-53.881Z/log.html

[Chris Hillery (ceejatec)]

> > - the jsoniq functions module doesn't need to be included
> Yes it does, it's used at line 67: jn:members($helper-host-ports)

My understanding is that the JSONiq functions module never needs to be included. Those functions are built-in to Zorba, like the F&O functions are. The .xq file for the module is included only for reference. Importing it explicitly just uses extra CPU cycles at compile time.

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2013-01-09T06-53-53.881Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Approve, 1 Needs Fixing.

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T18-02-06.862Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2013-01-09T18-02-06.862Z is finished. The final status was:

All tests succeeded!

[Zorba Build Bot (zorba-buildbot)]

Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1, Needs Fixing < 1, Pending < 1. Got: 2 Approve, 1 Needs Fixing.

[Matthias Brantner (matthias-brantner)]

review: Approve

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T22-34-54.165Z/log.html

[Zorba Build Bot (zorba-buildbot)]

The attempt to merge lp:~zorba-coders/zorba/f-nosql-db-trunk into lp:zorba failed. Below is the output from the failed tests.

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:275 (message):
  Validation queue job f-nosql-db-trunk-2013-01-09T22-34-54.165Z is finished.
  The final status was:

  2 tests did not succeed - changes not commited.

Error in read script: /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/f-nosql-db-trunk-2013-01-09T23-27-51.093Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job f-nosql-db-trunk-2013-01-09T23-27-51.093Z is finished. The final status was:

All tests succeeded!

=== modified file 'ChangeLog'
--- ChangeLog	2013-01-03 20:25:05 +0000
+++ ChangeLog	2013-01-04 16:25:28 +0000
@@ -9,6 +9,7 @@
 	when resolving URIs.
   * Can now specify jsoniq-strip-top-level-array option to parse-json() to
     strip the top-level array from JSON streams.
+  * New external module client for Oracle NoSQL Database.
   * New info-extraction module for querying concepts and entities in
 	natural language text.
   * Extended cast and castable expression to allow any simple target type
 
=== modified file 'modules/ExternalModules.conf'
--- modules/ExternalModules.conf	2012-12-26 07:31:06 +0000
+++ modules/ExternalModules.conf	2013-01-04 16:25:28 +0000
@@ -39,6 +39,7 @@
 http-client     bzr  lp:zorba/http-client-module      zorba-2.7
 image           bzr  lp:zorba/image-module
 languages       bzr  lp:zorba/languages-module        zorba-2.7
+oracle-nosql-db        bzr  lp:zorba/oracle-nosql-db-module 
 oauth           bzr  lp:zorba/oauth-module            zorba-2.7
 process         bzr  lp:zorba/process-module          zorba-2.7
 read-pdf        bzr  lp:zorba/read-pdf-module         zorba-2.7