[ 
https://issues.apache.org/jira/browse/AVRO-2234?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16678359#comment-16678359
 ] 

ASF GitHub Bot commented on AVRO-2234:
--------------------------------------

Fokko commented on issue #337: AVRO-2234 Use MappingProxyType, not ImmutableDict
URL: https://github.com/apache/avro/pull/337#issuecomment-436655347
 
 
   @kojiromike Can you create an account on https://issues.apache.org/? So I 
can assign you to the ticket, so you'll get the credit ;) Thanks!

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Use native MappingProxyType instead of hand-written ImmutableDict
> -----------------------------------------------------------------
>
>                 Key: AVRO-2234
>                 URL: https://issues.apache.org/jira/browse/AVRO-2234
>             Project: Avro
>          Issue Type: Improvement
>          Components: python
>            Reporter: Michael A. Smith
>            Priority: Major
>             Fix For: 1.9.0
>
>
> Per the rejection notice in 
> [PEP-416,|https://www.python.org/dev/peps/pep-0416/] there is no native 
> ImmutableDict in Python. However, there is a MappingProxyType that makes a 
> proxy to the mapping of a dictionary, and is fully native. For most purposes, 
> including avro's, it's a fully suitable replacement for a real ImmutableDict 
> implementation (except for a minor detail about JSON encoding that is easier 
> to solve and support than maintaining a complete implementation of 
> ImmutableDict.)
> I propose that we drop ImmutableDict and replace it with MappingProxyType.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to