Skip to content

[INLONG-7073][Sort] Support table level metrics for Apache Iceberg connector#7118

Merged
gong merged 2 commits intoapache:masterfrom
thesumery:7073
Jan 4, 2023
Merged

[INLONG-7073][Sort] Support table level metrics for Apache Iceberg connector#7118
gong merged 2 commits intoapache:masterfrom
thesumery:7073

Conversation

@thesumery
Copy link
Contributor

[INLONG-7073][Sort] Support table level metrics for Apache Iceberg connector

Prepare a Pull Request

Motivation

Support table level metrics for Apache Iceberg connector

@thesumery thesumery force-pushed the 7073 branch 2 times, most recently from fee0a24 to 0182473 Compare January 3, 2023 02:31
@gong
Copy link
Contributor

gong commented Jan 3, 2023

lost table level metric restore

@thesumery
Copy link
Contributor Author

lost table level metric restore

table level metric are in IcebergMultipleStreamWriter line 191,restore table metric at runtime

@gong
Copy link
Contributor

gong commented Jan 3, 2023

lost table level metric restore

table level metric are in IcebergMultipleStreamWriter line 191,restore table metric at runtime

@thesumery maybe you need call SinkTableMetricData#registerSubMetricsGroup for table level metric restore

@thesumery
Copy link
Contributor Author

registerSubMetricsGroup

It‘s different from other multiple sink connector.It's a 'IcebergMultipleStreamWriter' include many 'IcebergSingleStreamWriter',so SinkTableMetricData#registerSubMetricsGroup it not suitable for this.

@gong gong merged commit e305ae8 into apache:master Jan 4, 2023
liaorui pushed a commit to liaorui/inlong that referenced this pull request Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Sort] Support table level metrics for Apache Iceberg connector

5 participants