[
https://issues.apache.org/jira/browse/ORC-184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16008312#comment-16008312
]
ASF GitHub Bot commented on ORC-184:
------------------------------------
Github user majetideepak commented on the issue:
https://github.com/apache/orc/pull/120
You are right that we cannot make [Type]ColumnStatisticsImpl inherit
ColumnStatisticsImpl. That is why I introduced `InternalStatisticsImpl` in
ORC-185.
With that, you will just use `InternalStatisticsImpl<Type>` in your
templated [Type]ColumnWriterImpl.
I guess without the design for the ColumnWriter, it is hard to discuss at
this point.
Can you open a separate PR for your ColumnWrite code? Things will get more
clear then.
We can converge on the necessary overall changes.
> [C++] Refactor ColumnStatistics classes for writer
> --------------------------------------------------
>
> Key: ORC-184
> URL: https://issues.apache.org/jira/browse/ORC-184
> Project: ORC
> Issue Type: Sub-task
> Components: C++
> Reporter: Gang Wu
> Assignee: Gang Wu
>
> 1. Add setter functions to ColumnStatistics.
> 2. Refactor ColumnStatistics to reduce duplicate code.
> 3. Add more functions in Int128 and Decimal classes for
> DecimalColumnStatistics.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
