Uploaded image for project: 'Couchbase Java Client'
  1. Couchbase Java Client
  2. JCBC-58

The get() function in HTTPFuture and ViewFuture contain duplicate code

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1dp
    • Fix Version/s: 1.1-beta
    • Component/s: Core
    • Security Level: Public
    • Labels:
      None

      Description

      See JCBC-44 for more info.

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

        Activity

        Hide
        mikew Mike Wiederhold added a comment -

        Rags, why is this incomplete? This is a code refactoring task and has information for what needs to be refactored in the bug in the description. In our ViewFuture and HttpFuture classes their is duplicate code in the get function that should be removed.

        Show
        mikew Mike Wiederhold added a comment - Rags, why is this incomplete? This is a code refactoring task and has information for what needs to be refactored in the bug in the description. In our ViewFuture and HttpFuture classes their is duplicate code in the get function that should be removed.
        Hide
        ingenthr Matt Ingenthron added a comment -

        Rags had accidentally closed a number of these issues, including this one.

        Assigning to Michael to look into for refactoring before 1.1 release.

        Show
        ingenthr Matt Ingenthron added a comment - Rags had accidentally closed a number of these issues, including this one. Assigning to Michael to look into for refactoring before 1.1 release.
        Hide
        daschl Michael Nitschinger added a comment -
        Show
        daschl Michael Nitschinger added a comment - Tracked here: http://review.couchbase.org/#/c/22353/
        Hide
        daschl Michael Nitschinger added a comment -

        pushed to master, will be available in dp5.

        Show
        daschl Michael Nitschinger added a comment - pushed to master, will be available in dp5.

          People

          • Assignee:
            daschl Michael Nitschinger
            Reporter:
            mikew Mike Wiederhold
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes