Skip to content

Comments

[MINOR][DOC] Fix a typo 'lister' -> 'listener'#25377

Closed
lys0716 wants to merge 1 commit intoapache:masterfrom
lys0716:dev
Closed

[MINOR][DOC] Fix a typo 'lister' -> 'listener'#25377
lys0716 wants to merge 1 commit intoapache:masterfrom
lys0716:dev

Conversation

@lys0716
Copy link
Contributor

@lys0716 lys0716 commented Aug 7, 2019

What changes were proposed in this pull request?

Fix the typo in java doc.

How was this patch tested?

N/A

Signed-off-by: Yishuang Lu luystu@gmail.com

@srowen
Copy link
Member

srowen commented Aug 7, 2019

There are 3 more instances of this typo; can you fix all of them?
Any more typos you see?

Fix the typo in java doc.

N/A

Signed-off-by: Yishuang Lu <luystu@gmail.com>
@HyukjinKwon HyukjinKwon changed the title [MINOR][DOC] Fix typo in javadoc [MINOR][DOC] Fix a type 'lister' -> Aug 8, 2019
@HyukjinKwon HyukjinKwon changed the title [MINOR][DOC] Fix a type 'lister' -> [MINOR][DOC] Fix a type 'lister' -> 'listener' Aug 8, 2019
@HyukjinKwon
Copy link
Member

ok to test

@HyukjinKwon HyukjinKwon changed the title [MINOR][DOC] Fix a type 'lister' -> 'listener' [MINOR][DOC] Fix a typo 'lister' -> 'listener' Aug 8, 2019
Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Let's add some more similar instances next time. LGTM considering it's the first contribution.

@HyukjinKwon
Copy link
Member

I am merging this because related linter checks passed.

@SparkQA
Copy link

SparkQA commented Aug 8, 2019

Test build #108792 has finished for PR 25377 at commit 2ecaae0.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

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.

5 participants