Details

    • Type: Sub-task Sub-task
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: C++ - Library
    • Labels:
      None

      Description

      From c9cd369f4352472f62b1b1f23dd3bee122c97b14 Mon Sep 17 00:00:00 2001
      From: David Reiss <dreiss@fb.com>
      Date: Mon, 10 May 2010 19:17:11 +0000
      Subject: [PATCH 8/9] Thrift: Write-only PHP serialization protocol

      Summary:

      • It's a bit sloppy, but not too bad.
      • It will happily serialize data that cannot be represented in PHP (like
        maps with lists as keys).
      • No support for sets.
      • The hardest part is getting the "keys" right for a list.

      davejwatson: also added to Makefile.am

      Test Plan:
      Used it to serialize some not-totally-trivial data and PHP was happy
      with it.

      Revert Plan:
      ok


      lib/cpp/Makefile.am | 2 +
      lib/cpp/src/protocol/TPhpSerializeProtocol.cpp | 212 ++++++++++++++++++++++++
      lib/cpp/src/protocol/TPhpSerializeProtocol.h | 114 +++++++++++++
      3 files changed, 328 insertions, 0 deletions
      create mode 100644 lib/cpp/src/protocol/TPhpSerializeProtocol.cpp
      create mode 100644 lib/cpp/src/protocol/TPhpSerializeProtocol.h

        Activity

        Dave Watson created issue -
        Dave Watson made changes -
        Field Original Value New Value
        Attachment 0008-Thrift-Write-only-PHP-serialization-protocol.patch [ 12498010 ]
        Roger Meier made changes -
        Parent THRIFT-2422 [ 12703101 ]
        Issue Type Improvement [ 4 ] Sub-task [ 7 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Dave Watson
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development