Details

    • Type: Task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.0.12
    • Component/s: library
    • Security Level: Public
    • Labels:
      None
    • Environment:
      windows 7, linux

      Description

      Create documents with long strings inside them. Try to retrieve documents using get or multi-get. See that single callbacks or spooled callbacks do not always get called.

      Error is demonstrated best when creating documents with strings of greater than 1200 characters, with the error occurring more often the larger the strings.

      See following gist for test case:

      https://gist.github.com/benjamind/5247953

      Case should insert 4 documents with strings of 5000 characters each, then attempt to retrieve them 50 times. Will hang if error occurs.

      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        Hide
        trond Trond Norbye added a comment -

        I'm still seeing some problems (tried on linux today)

        Show
        trond Trond Norbye added a comment - I'm still seeing some problems (tried on linux today)
        Hide
        trond Trond Norbye added a comment -

        I'm currently testing http://review.couchbase.org/#/c/25517/ under valgrind to see if there is any memory-related problems (leaks etc). I encourage everyone to try out the version and report problems...

        Show
        trond Trond Norbye added a comment - I'm currently testing http://review.couchbase.org/#/c/25517/ under valgrind to see if there is any memory-related problems (leaks etc). I encourage everyone to try out the version and report problems...
        Hide
        koji kawamura koji kawamura added a comment -

        I've updated the couchnode to 0.0.12 and my problem with retrieving large documents has been solved. I am using basic CRUD operation, increment, View and design document management APIs. I don't see any other problem so far. Thanks for fixing this!

        Show
        koji kawamura koji kawamura added a comment - I've updated the couchnode to 0.0.12 and my problem with retrieving large documents has been solved. I am using basic CRUD operation, increment, View and design document management APIs. I don't see any other problem so far. Thanks for fixing this!
        Hide
        sam_jackson Sam Jackson added a comment -

        Hey guys, Just wondering if Trond's changes to libcouchbase ever made it into the official release? I'm seeing a similar issue to the one that Rod describes where the library will go into an endless loop in certain conditions under heavy load. It never releases the event loop to Node and as a result the application hangs without a warning.

        Show
        sam_jackson Sam Jackson added a comment - Hey guys, Just wondering if Trond's changes to libcouchbase ever made it into the official release? I'm seeing a similar issue to the one that Rod describes where the library will go into an endless loop in certain conditions under heavy load. It never releases the event loop to Node and as a result the application hangs without a warning.
        Hide
        ingenthr Matt Ingenthron added a comment -

        The issue tracked here is in 0.0.12, yes. That said, the current working branch has some more performance fixes. The one about releasing meory for command entries made a huge difference, IIRC. Here's the current list:

        4fbfbc4 (origin/master, origin/HEAD, gerrit/master) Add a couchnode benchmark sa
        65ec153 Release the memory allocated for command entries
        1fa3e3d Fixed test runner to display slash codes.
        fcdda10 JSCBC-24 Added v8 error catching
        4a2fc0d Removed unneccessary creation of new context.
        1d02572 (HEAD, master) Corrected missuse of Exception instead of Error.
        855bb4d JSCBC-25 Validate input parameters
        e39aa85 Add testcase for JSCBC-24
        ff353b1 (tag: 0.0.12) Release 0.0.12

        Show
        ingenthr Matt Ingenthron added a comment - The issue tracked here is in 0.0.12, yes. That said, the current working branch has some more performance fixes. The one about releasing meory for command entries made a huge difference, IIRC. Here's the current list: 4fbfbc4 (origin/master, origin/HEAD, gerrit/master) Add a couchnode benchmark sa 65ec153 Release the memory allocated for command entries 1fa3e3d Fixed test runner to display slash codes. fcdda10 JSCBC-24 Added v8 error catching 4a2fc0d Removed unneccessary creation of new context. 1d02572 (HEAD, master) Corrected missuse of Exception instead of Error. 855bb4d JSCBC-25 Validate input parameters e39aa85 Add testcase for JSCBC-24 ff353b1 (tag: 0.0.12) Release 0.0.12

          People

          • Assignee:
            trond Trond Norbye
            Reporter:
            benjamind Ben Delarre
          • Votes:
            1 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes