Mailing-List: contact dev-help@thrift.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@thrift.apache.org
From: elprans <git@git.apache.org>
To: dev@thrift.apache.org
Reply-To: dev@thrift.apache.org
Message-ID: <git-pr-1273-thrift@git.apache.org>
Subject: [GitHub] thrift pull request #1273: Fix decoding of strings in containers with py:dyn...
Content-Type: text/plain
Date: Tue, 23 May 2017 16:25:42 +0000 (UTC)
archived-at: Tue, 23 May 2017 16:25:45 -0000

GitHub user elprans opened a pull request:

    https://github.com/apache/thrift/pull/1273

    Fix decoding of strings in containers with py:dynamic and py:utf8strings

    _read_by_ttype and _write_by_ttype must be using the *element* spec
    and not the container spec when determining the correct read/write
    handler.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/elprans/thrift py-strings-in-containers

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/thrift/pull/1273.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1273
    
----
commit 6edf57c5073e4b452bb1752a9e493c9b779bb24a
Author: Elvis Pranskevichus <elvispranskevichus@pinterest.com>
Date:   2017-05-23T15:36:23Z

    Fix decoding of strings in containers with py:dynamic and py:utf8strings
    
    _read_by_ttype and _write_by_ttype must be using the *element* spec
    and not the container spec when determining the correct read/write
    handler.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---