roryqi comments

Results 406 comments of


                                            roryqi

[Improvement] Consider removing easyjson dependency due to sanction concerns

OK. I got it. I will check this.

[MINOR] Server heartbeat to coordinator while unregister shuffle

@maobaolong Could you rebase this pull request?

[#2212] feat(client,server,coordinator): Introduce ServiceVersion framework to support backward compatible and reduce BlockIdLayout message

I have already had server version, you can see Constants::SHUFFLE_SERVER_VERSION

[#2212] feat(client,server,coordinator): Introduce ServiceVersion framework to support backward compatible and reduce BlockIdLayout message

> @jerqi Thanks for this remind, after go through the code related to `SHUFFLE_SERVER_VERSION`, I have some primary knowledge about this. > > * SHUFFLE_SERVER_VERSION is used as a tag...

[#2282][FOLLOWUP] test: Add UT for DelegationRssShuffleManager#getReader

> The failed test caused by there is no specific `getReader` method, in `spark 3.0`, but it exists in other minor version of `spark 3.x` You should use reflection to...

[#5803] feat(docker-image): Change username in `hive` Dockerfile from datastrato to gravitino

CI seems to pass. Ok for me.

[#5803] feat(docker-image): Change username in `hive` Dockerfile from datastrato to gravitino

Is this ready to review?

[#5803] feat(docker-image): Change username in `hive` Dockerfile from datastrato to gravitino

LGTM if CI passed.

[Improvement] Add timestamp information to metrics

@Abyss-lord Are working on this issue? Maybe we want to include this to the 0.9.

[#6133] Improvement(core): Supports get Fileset schema location in the AuthorizationUtils

@Abyss-lord Will you continue this PR?