Junfan Zhang comments

Results 434 comments of


                                            Junfan Zhang

[FEATURE] Add the integration with existing Spark tests on rust based server

> Hi @zuston do we have a default config.toml which contains basic config about shuffle server, I will use Java to start rust program. If not I will create a...

[FEATURE] Add the integration with existing Spark tests on rust based server

The `partitionId` in RequireBufferContext won't be used in the process of requiring buffer, which is only a placeholder. Please ignore this @wenlongbrother

[#1608][part-6] improvement(spark): verify the sent blocks count

PTAL @jerqi

[#1608][part-6] improvement(spark): verify the sent blocks count

> PTAL @jerqi Gentle ping @jerqi

fix:read inconsistent when local file read handler init before flushing event to disk

I don't see the root cause of this bug. @dingshun3016 Could you help add test case to simulate this or add more description

fix:read inconsistent when local file read handler init before flushing event to disk

Thanks for adding detailed logs, I have understood your point, the root cause is to partition shuffle reader handler is cached. After the memory data is flushed into local file,...

fix:read inconsistent when local file read handler init before flushing event to disk

Please create the issue to track this bug @dingshun3016

[#1673] fix(K8S): Fix the deployment of stable version K8S cluster

cc @rickyma @zhengchenyu This is a break change for your deployment, please take a look

[#1675][FOLLOWUP] fix(test): Fix flaky tests which may cause port conflicts

LGTM. Please update the PR title to describe what's this PR changed rather than coarse grained desp.

[#1675][FOLLOWUP] fix(test): Fix flaky tests which may cause port conflicts

> Maybe we should commit this to branch 0.9, too. Done