You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hudi.apache.org by "Sagar Sumit (Jira)" <ji...@apache.org> on 2022/05/24 15:51:00 UTC
[jira] [Closed] (HUDI-4052) Move AWS Cloudwatch metric configs to hudi-client-common instead of hudi-aws
[ https://issues.apache.org/jira/browse/HUDI-4052?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sagar Sumit closed HUDI-4052.
-----------------------------
Resolution: Fixed
> Move AWS Cloudwatch metric configs to hudi-client-common instead of hudi-aws
> ----------------------------------------------------------------------------
>
> Key: HUDI-4052
> URL: https://issues.apache.org/jira/browse/HUDI-4052
> Project: Apache Hudi
> Issue Type: Task
> Reporter: Sagar Sumit
> Assignee: Sagar Sumit
> Priority: Critical
> Labels: pull-request-available
> Fix For: 0.11.1
>
>
> Currently, HoodieMetricsCloudWatchConfig is part of hudi-aws module, while HoodieMetricConfig is part of hudi-client-common. The builder of HoodieMetricConfig uses HoodieMetricsCloudWatchConfig. When we want to use hudi write client in other engines (presto/trino), we need to include hudi-aws module because HoodieMetricsConfig initialization falls in that path. But, this bloats the presto/trino bundle, which we want to keep as light as possible.
> We should try to decouple the two. Let all metric configs be in client-common (e.g. prometheus and datadog configs are already there). This way we can exclude hudi-aws from the bundles.
--
This message was sent by Atlassian Jira
(v8.20.7#820007)