Skip to content

Comments

[MINOR] Add @since for DSv2 API#35935

Closed
pan3793 wants to merge 1 commit intoapache:masterfrom
pan3793:minor
Closed

[MINOR] Add @since for DSv2 API#35935
pan3793 wants to merge 1 commit intoapache:masterfrom
pan3793:minor

Conversation

@pan3793
Copy link
Member

@pan3793 pan3793 commented Mar 22, 2022

What changes were proposed in this pull request?

Add missing since for DSv2 API

Why are the changes needed?

Let users know which version of the API was introduced.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

The change does not touch the code.

@pan3793
Copy link
Member Author

pan3793 commented Mar 22, 2022

cc @dongjoon-hyun

@HyukjinKwon
Copy link
Member

Merged to master and branch-3.3.

cc @HeartSaVioR too FYI

HyukjinKwon pushed a commit that referenced this pull request Mar 23, 2022
### What changes were proposed in this pull request?

Add missing `since` for DSv2 API

### Why are the changes needed?

Let users know which version of the API was introduced.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

The change does not touch the code.

Closes #35935 from pan3793/minor.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit f73d528)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@HeartSaVioR
Copy link
Contributor

Nice catch!

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants