Merge lp:~zorba-coders/zorba/compile_queryplan into lp:zorba

Proposed by William Candillon
Status: Merged
Approved by: William Candillon
Approved revision: 10675
Merged at revision: 10680
Proposed branch: lp:~zorba-coders/zorba/compile_queryplan
Merge into: lp:zorba
Diff against target: 30 lines (+3/-3)
2 files modified
bin/zorbacmdproperties.txt (+1/-1)
bin/zorbacmdproperties_base.h (+2/-2)
To merge this branch: bzr merge lp:~zorba-coders/zorba/compile_queryplan
Reviewer Review Type Date Requested Status
Matthias Brantner Approve
William Candillon Approve
Review via email: mp+94733@code.launchpad.net

Commit message

Remove conflicting shortcut -e for --execute-plan option.

Description of the change

Remove conflicting shortcut -e for --execute-plan option.

To post a comment you must log in.
Revision history for this message
William Candillon (wcandillon) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job compile_queryplan-2012-02-27T11-50-03.177Z is finished. The final status was:

All tests succeeded!

Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

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

Revision history for this message
Matthias Brantner (matthias-brantner) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

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

CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274 (message):
  Validation queue job compile_queryplan-2012-02-27T15-23-58.529Z 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

Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

Validation queue job compile_queryplan-2012-02-27T18-40-06.124Z is finished. The final status was:

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bin/zorbacmdproperties.txt'
--- bin/zorbacmdproperties.txt 2012-02-20 10:10:44 +0000
+++ bin/zorbacmdproperties.txt 2012-02-27 09:10:24 +0000
@@ -36,4 +36,4 @@
36("stop-words", po::value<std::vector<std::string> >(), "Mapping specifying a stop-words URI to another.")36("stop-words", po::value<std::vector<std::string> >(), "Mapping specifying a stop-words URI to another.")
37("thesaurus", po::value<std::vector<std::string> >(), "Mapping specifying a thesaurus URI to another.")37("thesaurus", po::value<std::vector<std::string> >(), "Mapping specifying a thesaurus URI to another.")
38("compile-plan,c", "Output the query plan as binary.")38("compile-plan,c", "Output the query plan as binary.")
39("execute-plan,e", "Take a query plan as binary and execute it. Binary query plans can be generated using the --compile-plan option.")39("execute-plan", "Take a query plan as binary and execute it. Binary query plans can be generated using the --compile-plan option.")
4040
=== modified file 'bin/zorbacmdproperties_base.h'
--- bin/zorbacmdproperties_base.h 2012-02-20 10:10:44 +0000
+++ bin/zorbacmdproperties_base.h 2012-02-27 09:10:24 +0000
@@ -308,7 +308,7 @@
308 else if (strcmp (*argv, "--compile-plan") == 0 || strncmp (*argv, "-c", 2) == 0) {308 else if (strcmp (*argv, "--compile-plan") == 0 || strncmp (*argv, "-c", 2) == 0) {
309 theCompilePlan = true;309 theCompilePlan = true;
310 }310 }
311 else if (strcmp (*argv, "--execute-plan") == 0 || strncmp (*argv, "-e", 2) == 0) {311 else if (strcmp (*argv, "--execute-plan") == 0) {
312 theExecutePlan = true;312 theExecutePlan = true;
313 }313 }
314 else if (strcmp (*argv, "--") == 0) {314 else if (strcmp (*argv, "--") == 0) {
@@ -365,7 +365,7 @@
365"--stop-words\nMapping specifying a stop-words URI to another.\n\n"365"--stop-words\nMapping specifying a stop-words URI to another.\n\n"
366"--thesaurus\nMapping specifying a thesaurus URI to another.\n\n"366"--thesaurus\nMapping specifying a thesaurus URI to another.\n\n"
367"--compile-plan, -c\nOutput the query plan as binary.\n\n"367"--compile-plan, -c\nOutput the query plan as binary.\n\n"
368"--execute-plan, -e\nTake a query plan as binary and execute it. Binary query plans can be generated using the --compile-plan option.\n\n"368"--execute-plan\nTake a query plan as binary and execute it. Binary query plans can be generated using the --compile-plan option.\n\n"
369;369;
370 }370 }
371371

Subscribers

People subscribed via source and target branches