[
https://issues.apache.org/jira/browse/IMPALA-11869?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Daniel Becker updated IMPALA-11869:
-----------------------------------
Description: The change _IMPALA-11645: Remove PrintThriftEnum functions in
debug-util.cc_ introduced the function {{PrintValue()}} which we use to convert
Thrift types to strings. This function now uses {{{}operator<<{}}}, which is
provided for the generated thrift types. However, Thrift also generates a
{{to_string()}} function that is overloaded for Thrift types. It would be more
efficient to use this instead of involving streams with {{{}operator<<{}}}.
(was: The change_ IMPALA-11645: Remove PrintThriftEnum functions in
debug-utils.cc_ introduced the function {{PrintValue()}} which we use to
convert Thrift types to strings. This function now uses {{operator<<}}, which
is provided for the generated thrift types. However, Thrift also generates a
{{to_string()}} function that is overloaded for Thrift types. It would be more
efficient to use this instead of involving streams with {{operator<<}}.
)
> Use to_string() in PrintValue for printing thrift types
> -------------------------------------------------------
>
> Key: IMPALA-11869
> URL: https://issues.apache.org/jira/browse/IMPALA-11869
> Project: IMPALA
> Issue Type: Improvement
> Components: Backend
> Reporter: Daniel Becker
> Assignee: Daniel Becker
> Priority: Minor
>
> The change _IMPALA-11645: Remove PrintThriftEnum functions in debug-util.cc_
> introduced the function {{PrintValue()}} which we use to convert Thrift types
> to strings. This function now uses {{{}operator<<{}}}, which is provided for
> the generated thrift types. However, Thrift also generates a {{to_string()}}
> function that is overloaded for Thrift types. It would be more efficient to
> use this instead of involving streams with {{{}operator<<{}}}.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
