Uploaded image for project: 'Java Couchbase JVM Core'
  1. Java Couchbase JVM Core
  2. JVMCBC-510

Parse bucket configuration failure

    XMLWordPrintable

Details

    • 1

    Description

      After the addition of bucket capability "snappy", parsing of bucket configuration throws:

      InvalidFormatException: Can not construct instance of com.couchbase.client.core.config.BucketCapabilities from String value ("snappy"): value not one of declared Enum instance names: [nodesExt, dcp, cbhello, xattr, couchapi, cccp, touch, xdcrCheckpointing]

      Attachments

        Issue Links

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

          Activity

            Did you actually see what you've pasted here Abdullah Alamoudi [X]? I just added a unit test and I think it's fine after I fixed the xdcrxdcrCheckpointing bug which both Michael Blow and I noticed.

            ingenthr Matt Ingenthron added a comment - Did you actually see what you've pasted here Abdullah Alamoudi [X] ? I just added a unit test and I think it's fine after I fixed the xdcrxdcrCheckpointing bug which both Michael Blow and I noticed.

            I've verified this another couple ways, seems okay once the typo is fixed. If you saw the above^, I can't see how that'd be on core 1.5.5, but I admit, I might be missing something.

            Also, note that we have a method of transforming the configuration before interpretation if required. I think that too may have been used to workaround this. Leaving note to future bug browsers.

            ingenthr Matt Ingenthron added a comment - I've verified this another couple ways, seems okay once the typo is fixed. If you saw the above^, I can't see how that'd be on core 1.5.5, but I admit, I might be missing something. Also, note that we have a method of transforming the configuration before interpretation if required. I think that too may have been used to workaround this. Leaving note to future bug browsers.

            My fault Abdullah Alamoudi [X], I do see that it's broken. For the developer build, I've asked Ajit Yagaty [X] to remove it, per earlier plan discussed with Dave Finlay.

            ingenthr Matt Ingenthron added a comment - My fault Abdullah Alamoudi [X] , I do see that it's broken. For the developer build, I've asked Ajit Yagaty [X] to remove it, per earlier plan discussed with Dave Finlay .

            Thanks Matt Ingenthron,

            Was away from my computer for sometime

            Abdullah.Alamoudi Abdullah Alamoudi [X] (Inactive) added a comment - Thanks Matt Ingenthron , Was away from my computer for sometime

            For what it's worth, the underlying lib (Jackson) is the slightly inflexible one when object mapping. There is a change though that should allow it to be more flexible.

            https://fasterxml.github.io/jackson-databind/javadoc/2.8/com/fasterxml/jackson/databind/DeserializationFeature.html#READ_UNKNOWN_ENUM_VALUES_AS_NULL

            ingenthr Matt Ingenthron added a comment - For what it's worth, the underlying lib (Jackson) is the slightly inflexible one when object mapping. There is a change though that should allow it to be more flexible. https://fasterxml.github.io/jackson-databind/javadoc/2.8/com/fasterxml/jackson/databind/DeserializationFeature.html#READ_UNKNOWN_ENUM_VALUES_AS_NULL
            michael.blow Michael Blow added a comment -

            Removing the DP label, as Analytics has a workaround in place for the DP build.  If there are other uses of the SDK that require this for the Vulcan DP build, please restore the DP label.

            michael.blow Michael Blow added a comment - Removing the DP label, as Analytics has a workaround in place for the DP build.  If there are other uses of the SDK that require this for the Vulcan DP build, please restore the DP label.

            People

              daschl Michael Nitschinger
              Abdullah.Alamoudi Abdullah Alamoudi [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty