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

[BP MB-39974] - Support 6.0.x to 6.6 upgrade of handlers using N1qlQuery() Beta API

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 6.5.1
    • Fix Version/s: 6.6.1
    • Component/s: eventing
    • Triage:
      Untriaged
    • Story Points:
      1
    • Is this a Regression?:
      Unknown

      Description

      If customer is using N1QL in 5.5.x or 6.0.x - due to the need to do close(), and if they had dropped down to N1qlQuery() class the change to N1QL() call, they will be forced to undeploy the handler before upgrading to 6.5.1. This presents an unacceptable challenge if the handlers were taking actions with side effects. While admittedly these features were Beta, we still can't not have a working recipe to allow upgrade, as there are several significant users online today. This bug is to ensure we have a working path for these users to upgrade to 6.5.1 without having to reprocess all their data (assuming MB-39079 can be solved)

        Attachments

          Issue Links

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

            Activity

            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-6.6.1-9012 contains eventing commit a55e7a0 with commit message:
            MB-40637: Added N1QLQuery support

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9012 contains eventing commit a55e7a0 with commit message: MB-40637 : Added N1QLQuery support
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-6.6.0-7914 contains eventing commit a55e7a0 with commit message:
            MB-40637: Added N1QLQuery support

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.0-7914 contains eventing commit a55e7a0 with commit message: MB-40637 : Added N1QLQuery support
            Hide
            vikas.chaudhary Vikas Chaudhary added a comment -

            Vinayaka Kamath Jeelan Poola can we add qe notes and the what is the behaviour without fix if we won't undeploy the handler?

            Show
            vikas.chaudhary Vikas Chaudhary added a comment - Vinayaka Kamath Jeelan Poola  can we add qe notes and the what is the behaviour without fix if we won't undeploy the handler?
            Hide
            siri Sriram Melkote (Inactive) added a comment -

            Hi Vikas Chaudhary - without this fix, a handler that uses N1qlQuery will stop working on nodes that are upgraded. It will throw an error on lines of "ReferenceError: N1qlQuery is not defined" when it hits the line that calls N1qlQuery. With the fix, it will run without errors same way in older and newer nodes.

            Show
            siri Sriram Melkote (Inactive) added a comment - Hi Vikas Chaudhary - without this fix, a handler that uses N1qlQuery will stop working on nodes that are upgraded. It will throw an error on lines of "ReferenceError: N1qlQuery is not defined" when it hits the line that calls N1qlQuery. With the fix, it will run without errors same way in older and newer nodes.
            Hide
            vikas.chaudhary Vikas Chaudhary added a comment -

            Verified with online upgrade from  6.0.4-3082 -> 6.6.0-7897 (ReferenceError) -> 6.6.0-7918 (worked fine)

            Jeelan Poola Sriram Melkote One more suggestion to add depreciation message as a warning after the upgrade as we will not support 2 syntaxes in Cheshire-Cat. Also, we should revert this in CC. 

            Show
            vikas.chaudhary Vikas Chaudhary added a comment - Verified with online upgrade from  6.0.4-3082 -> 6.6.0-7897 (ReferenceError) -> 6.6.0-7918 (worked fine) Jeelan Poola Sriram Melkote  One more suggestion to add depreciation message as a warning after the upgrade as we will not support 2 syntaxes in Cheshire-Cat. Also, we should revert this in CC. 
            Hide
            jeelan.poola Jeelan Poola added a comment -

            Vikas Chaudhary Logged MB-41125 to track your suggestions.

            Show
            jeelan.poola Jeelan Poola added a comment - Vikas Chaudhary Logged MB-41125 to track your suggestions.

              People

              Assignee:
              vinayaka.kamath Vinayaka Kamath
              Reporter:
              jeelan.poola Jeelan Poola
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes

                    PagerDuty