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

Zheng Hu commented on FLINK-27543:
----------------------------------

I think a better approach is hiding the `fileStatsExtractor` or 
`StatsExtractor` inside the writer which was created by the file format's 
writer factory. 

Let me publish a PR for this.

> Introduce StatsProducer to refactor code in DataFileWriter 
> -----------------------------------------------------------
>
>                 Key: FLINK-27543
>                 URL: https://issues.apache.org/jira/browse/FLINK-27543
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table Store
>            Reporter: Jingsong Lee
>            Priority: Major
>             Fix For: table-store-0.2.0
>
>
> Lots of `fileStatsExtractor == null` looks bad.
> I think we can have a `StatsProducer` to unify `StatsExtractor` and 
> `StatsCollector`. To reduce caller complexity.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to