Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.4
    • Component/s: None
    • Security Level: Public
    • Labels:
      None

      Description

      http://couchbase.com/forums/thread/igbinary-php-client reports

      I'm a long time memory memcached user, and I always compiled the memcached-php extension to have igbinary. I followed the instructions to get source and unfortunately I was not able to pull the repo. Is there a version that has igbinary configured?
      phpinfo reports the igbinary serializer is not avaliable in your default version.
      Am I better of using the stock serializer?
      Memcached::OPT_SERIALIZER
      Specifies the serializer to use for serializing non-scalar values. The valid serializers are Memcached::SERIALIZER_PHP or Memcached::SERIALIZER_IGBINARY. The latter is supported only when memcached is configured with --enable-memcached-igbinary option and the igbinary extension is loaded.
      Type: integer, default: Memcached::SERIALIZER_PHP.

      we should look into enabling igbinary for the pre-1.8 SDKs as well it's feasibility for the >= 1.8 SDKs that are based on libcouchbase.

      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 -

        http://review.couchbase.org/25701

        This can go into 1.1.4 as an experimental feature. We don't add any new functions etc in order to support this, but just a new configuration parameter

        Show
        trond Trond Norbye added a comment - http://review.couchbase.org/25701 This can go into 1.1.4 as an experimental feature. We don't add any new functions etc in order to support this, but just a new configuration parameter
        Hide
        ingenthr Matt Ingenthron added a comment -

        This would be nice to have, if we can find a good way to enable it optionally.

        Show
        ingenthr Matt Ingenthron added a comment - This would be nice to have, if we can find a good way to enable it optionally.
        Hide
        mnunberg Mark Nunberg added a comment -

        There is no igbinary support built-in. We use the default serializer. Determine if there's a need for this still.

        Show
        mnunberg Mark Nunberg added a comment - There is no igbinary support built-in. We use the default serializer. Determine if there's a need for this still.
        Hide
        ingenthr Matt Ingenthron added a comment -

        Mark: Please determine if this issue is still valid and help me triage it into the appropriate release. I've assigned it to you for triage, not necessarily to fix it.

        Show
        ingenthr Matt Ingenthron added a comment - Mark: Please determine if this issue is still valid and help me triage it into the appropriate release. I've assigned it to you for triage, not necessarily to fix it.

          People

          • Assignee:
            trond Trond Norbye
            Reporter:
            jan Jan Lehnardt (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes