Skip to content

Conversation

@rickyma
Copy link
Contributor

@rickyma rickyma commented Feb 28, 2024

What changes were proposed in this pull request?

Refresh timestamp when sending SendShuffleDataRequest.

Why are the changes needed?

A follow-up PR for: #1534

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing UTs.

@rickyma
Copy link
Contributor Author

rickyma commented Feb 28, 2024

@jerqi @zuston PTAL

@github-actions
Copy link

Test Results

2 312 files  ±0  2 312 suites  ±0   4h 37m 10s ⏱️ - 1m 2s
  823 tests ±0    822 ✅ ±0   1 💤 ±0  0 ❌ ±0 
9 693 runs  ±0  9 679 ✅ ±0  14 💤 ±0  0 ❌ ±0 

Results for commit 22a1d7f. ± Comparison against base commit d4c0fc5.

@jerqi jerqi changed the title [#1472][part-5][FOLLOWUP] Fix Netty transport time when sending shuffle data requests [#1472][part-6][FOLLOWUP] Fix Netty transport time when sending shuffle data requests Feb 29, 2024
Copy link
Contributor

@jerqi jerqi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@zuston zuston merged commit 9f4ccde into apache:master Feb 29, 2024
@rickyma rickyma deleted the issue-1472-part5-followup branch May 5, 2024 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants