[ https://issues.apache.org/jira/browse/THRIFT-3214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14611058#comment-14611058 ]
ASF GitHub Bot commented on THRIFT-3214: ---------------------------------------- Github user jparise commented on a diff in the pull request: https://github.com/apache/thrift/pull/535#discussion_r33729457 --- Diff: compiler/cpp/src/generate/t_erl_generator.cc --- @@ -55,6 +55,7 @@ class t_erl_generator : public t_generator { out_dir_base_ = "gen-erl"; legacy_names_ = (parsed_options.find("legacynames") != parsed_options.end()); + use_maps_ = (parsed_options.find("usemaps") != parsed_options.end()); --- End diff -- I think `maps` would be a more consistent option name. Similar existing options are `slots` (for Python) and `templates` (for C++), which don't include a **use** prefix. > Add Erlang option for using maps instead of dicts > ------------------------------------------------- > > Key: THRIFT-3214 > URL: https://issues.apache.org/jira/browse/THRIFT-3214 > Project: Thrift > Issue Type: Improvement > Components: Erlang - Compiler > Reporter: Michael Oliver > Assignee: Michael Oliver > Priority: Minor > > Erlang OTP 18 introduces significant performance improvements when using maps > instead of dicts. Here is a benchmark showing the gains: > https://gist.github.com/BinaryMuse/bb9f2cbf692e6cfa4841 > Thrift should support an option for the Erlang compiler to produce maps > instead of dicts, to take advantage of these performance improvements. -- This message was sent by Atlassian JIRA (v6.3.4#6332)