[ 
https://issues.apache.org/jira/browse/HUDI-3532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Ethan Guo updated HUDI-3532:
----------------------------
    Description: 
The api {{getColumnStats}} is not supported in FileSystemBackedTableMetadata:
{code:java}
@Override
  public Map<Pair<String, String>, HoodieMetadataColumnStats> 
getColumnStats(final List<Pair<String, String>> partitionNameFileNameList, 
final String columnName)
      throws HoodieMetadataException {
    throw new HoodieMetadataException("Unsupported operation: 
getColumnsStats!");
  } {code}
It's better to support column stats without metadata table from 
FileSystemBackedTableMetadata as well to unify the logic around col stats and 
reduce the special-cased logic for col stats between metadata table vs file 
system backed metadata.

> Refactor FileSystemBackedTableMetadata and related classes to support 
> getColumnStats directly
> ---------------------------------------------------------------------------------------------
>
>                 Key: HUDI-3532
>                 URL: https://issues.apache.org/jira/browse/HUDI-3532
>             Project: Apache Hudi
>          Issue Type: Task
>            Reporter: Ethan Guo
>            Priority: Major
>             Fix For: 0.11.0
>
>
> The api {{getColumnStats}} is not supported in FileSystemBackedTableMetadata:
> {code:java}
> @Override
>   public Map<Pair<String, String>, HoodieMetadataColumnStats> 
> getColumnStats(final List<Pair<String, String>> partitionNameFileNameList, 
> final String columnName)
>       throws HoodieMetadataException {
>     throw new HoodieMetadataException("Unsupported operation: 
> getColumnsStats!");
>   } {code}
> It's better to support column stats without metadata table from 
> FileSystemBackedTableMetadata as well to unify the logic around col stats and 
> reduce the special-cased logic for col stats between metadata table vs file 
> system backed metadata.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to