Skip to content

[CORE][MINOR] Correct spelling for RPC in log#39200

Closed
tedyu wants to merge 1 commit intoapache:masterfrom
tedyu:trans-rpc
Closed

[CORE][MINOR] Correct spelling for RPC in log#39200
tedyu wants to merge 1 commit intoapache:masterfrom
tedyu:trans-rpc

Conversation

@tedyu
Copy link
Contributor

@tedyu tedyu commented Dec 24, 2022

What changes were proposed in this pull request?

This PR corrects spelling mistake for RPC in log for sendRpc.
Similar error in context.R is also fixed.

Why are the changes needed?

The spelling mistake confuses users.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Existing test suite

@github-actions github-actions bot added the CORE label Dec 24, 2022
@tedyu
Copy link
Contributor Author

tedyu commented Dec 24, 2022

cc @mridulm

@tedyu
Copy link
Contributor Author

tedyu commented Dec 24, 2022

cc @srowen

@srowen
Copy link
Member

srowen commented Dec 24, 2022

Any other typos like this?
The same issue appears in context.R; we could fix it here.

@tedyu
Copy link
Contributor Author

tedyu commented Dec 24, 2022

@srowen
I have included context.R in the PR.

@srowen
Copy link
Member

srowen commented Dec 24, 2022

Merged to master

@srowen srowen closed this in 02fdefd Dec 24, 2022
@tedyu
Copy link
Contributor Author

tedyu commented Dec 24, 2022

Thanks, @srowen
Happy Christmas !

@mridulm
Copy link
Contributor

mridulm commented Dec 27, 2022

Thanks for fixing this @tedyu, and thanks for merging during holidays @srowen :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments