Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AMORO-1906]Refactor code of "Table" and "Catalog" to support Paimon format integration #1960

Merged
merged 39 commits into from
Oct 10, 2023

polish code

a5beb36
Select commit
Loading
Failed to load commit list.
Merged

[AMORO-1906]Refactor code of "Table" and "Catalog" to support Paimon format integration #1960

polish code
a5beb36
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Oct 9, 2023 in 2s

46.99% of diff hit (target 50.80%)

View this Pull Request on Codecov

46.99% of diff hit (target 50.80%)

Annotations

Check warning on line 35 in ams/server/src/main/java/com/netease/arctic/server/catalog/InternalMixedCatalogImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/catalog/InternalMixedCatalogImpl.java#L35 

Added line #L35 was not covered by tests

Check warning on line 54 in ams/server/src/main/java/com/netease/arctic/server/catalog/MixedHiveCatalogImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/catalog/MixedHiveCatalogImpl.java#L54 

Added line #L54 was not covered by tests

Check warning on line 52 in ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java#L50-L52 

Added lines #L50 - L52 were not covered by tests

Check warning on line 57 in ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java#L57 

Added line #L57 was not covered by tests

Check warning on line 80 in ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/catalog/PaimonServerCatalog.java#L76-L80 

Added lines #L76 - L80 were not covered by tests

Check warning on line 89 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L89 

Added line #L89 was not covered by tests

Check warning on line 101 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L91-L101 

Added lines #L91 - L101 were not covered by tests

Check warning on line 103 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L103 

Added line #L103 was not covered by tests

Check warning on line 105 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L105 

Added line #L105 was not covered by tests

Check warning on line 109 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L107-L109 

Added lines #L107 - L109 were not covered by tests

Check warning on line 111 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L111 

Added line #L111 was not covered by tests

Check warning on line 127 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L113-L127 

Added lines #L113 - L127 were not covered by tests

Check warning on line 131 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L129-L131 

Added lines #L129 - L131 were not covered by tests

Check warning on line 135 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L133-L135 

Added lines #L133 - L135 were not covered by tests

Check warning on line 141 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L139-L141 

Added lines #L139 - L141 were not covered by tests

Check warning on line 144 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L143-L144 

Added lines #L143 - L144 were not covered by tests

Check warning on line 146 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L146 

Added line #L146 was not covered by tests

Check warning on line 148 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L148 

Added line #L148 was not covered by tests

Check warning on line 150 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L150 

Added line #L150 was not covered by tests

Check warning on line 153 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L153 

Added line #L153 was not covered by tests

Check warning on line 167 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L155-L167 

Added lines #L155 - L167 were not covered by tests

Check warning on line 176 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L169-L176 

Added lines #L169 - L176 were not covered by tests

Check warning on line 181 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L180-L181 

Added lines #L180 - L181 were not covered by tests

Check warning on line 184 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L184 

Added line #L184 was not covered by tests

Check warning on line 186 in ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

ams/server/src/main/java/com/netease/arctic/server/dashboard/MixedAndIcebergTableDescriptor.java#L186 

Added line #L186 was not covered by tests
View more details on Codecov