Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-5607

docloader fails to run at all in 2.0 builds leading to DP5

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta
    • Fix Version/s: 2.0-beta, 2.0
    • Component/s: tools
    • Security Level: Public
    • Labels:
      None
    • Environment:
      Version: 2.0.0 community edition (build-1269)
      CentOS 5.x, 64-bit

      Description

      Running the docloader without any arguments I expected to get a usage or something nice. Instead, I received:

      1. /opt/couchbase/bin/docloader
        File "/opt/couchbase/lib/python/docloader", line 90
        with open(os.path.join(basedir, item), 'r') as fp:
        ^
        SyntaxError: invalid syntax
      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        Hide
        pavelpaulau Pavel Paulau added a comment -
        Show
        pavelpaulau Pavel Paulau added a comment - This also must be merged to branch "couchbase-181": https://github.com/couchbase/couchbase-python-client/commit/b31ab60ff79601683d25c0c2feead75507060894
        Hide
        farshid Farshid Ghods (Inactive) added a comment -

        the fix to python sdk code you mentioned is one line change to use .join or % instead of .format

        you can merge that code to branch-1.8.1 and still have the doc loader run the view once

        the other three changes you posted look good to me so let's merge them before recent builds

        Show
        farshid Farshid Ghods (Inactive) added a comment - the fix to python sdk code you mentioned is one line change to use .join or % instead of .format you can merge that code to branch-1.8.1 and still have the doc loader run the view once the other three changes you posted look good to me so let's merge them before recent builds
        Show
        pavelpaulau Pavel Paulau added a comment - Done. http://review.couchbase.org/#change,17933 http://review.couchbase.org/#change,17934
        Hide
        pavelpaulau Pavel Paulau added a comment - - edited

        Currently using sdk branch "couchbase-181" it works:
        – via cli
        – via setting menu

        It doesn't work before cluster initialization.

        @Farshid
        Is it ok?

        Show
        pavelpaulau Pavel Paulau added a comment - - edited Currently using sdk branch "couchbase-181" it works: – via cli – via setting menu It doesn't work before cluster initialization. @Farshid Is it ok?
        Hide
        farshid Farshid Ghods (Inactive) added a comment -

        yes that works.

        Show
        farshid Farshid Ghods (Inactive) added a comment - yes that works.

          People

          • Assignee:
            pavelpaulau Pavel Paulau
            Reporter:
            ingenthr Matt Ingenthron
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes