FANNG comments

Results 194 comments of


                                            FANNG

[#2921] fix(catalog-lakehouse-iceberg): Add width param to bucket and truncate functions in Gravitino SortOrder

LGTM except the comment, @mchades do you have time to review again?

[Improvement] Suggestion for Handling Stale Metrics in Gravitino

thanks for reporting this issue.

[Improvement] Suggestion for Handling Stale Metrics in Gravitino

@TEOTEO520 , #3341 propose a way to compuate Pxx according to the time sliding window, you could check it when you are free. I'm not sure whether to make it...

[Improvement] Spark Connector Need DelegationTokenProvider for k8s deployment

cc @danhuawang ,could our test cluster cover the scene?

[#3259] feat(spark-connector): Support atomic create or replace Iceberg table

> Hi @FANNG1 could you help review this PR? Thank you. ok

[Bug report] Doris table operation integration test failed occasionally

seems couldn't reproduce on main branch, but easy to reproduce on PR: https://github.com/datastrato/gravitino/pull/3307

[Bug report] Doris table operation integration test failed occasionally

seems not the same problem, the problem you mentioned is alter operation does not takes affect. but in this case, alter operation is not allowed for `state is not NORMAL`

`java.lang.IncompatibleClassChangeError: Implementing class` error occured when ran IT locally

> at org.junit.jupiter.engine.extension.TimeoutExtension.interceptBeforeAllMethod(TimeoutExtension.java:68) does it still exist in the latest main branch?

`java.lang.IncompatibleClassChangeError: Implementing class` error occured when ran IT locally

`IncompatibleClassChangeError`, try change a java sdk?

[#4393] improvement(docs): Refine authentication information about Iceberg rest catalog and Add IT for Kerberos Hive backend.

> Actually I have a concern about the rest service configuration style. The last segment isn't camel style, is it ok? There are mixed styles in Gravitino and IcebergRESTServer now...