Skip to content

[MINOR] clean up style for storage param setters in ALS#13480

Closed
MLnick wants to merge 1 commit intoapache:masterfrom
MLnick:als-param-minor-cleanup
Closed

[MINOR] clean up style for storage param setters in ALS#13480
MLnick wants to merge 1 commit intoapache:masterfrom
MLnick:als-param-minor-cleanup

Conversation

@MLnick
Copy link
Contributor

@MLnick MLnick commented Jun 2, 2016

Clean up style for param setter methods in ALS to match standard style and the other setter in class (this is an artefact of one of my previous PRs that wasn't cleaned up).

How was this patch tested?

Existing tests - no functionality change.

@mengxr
Copy link
Contributor

mengxr commented Jun 2, 2016

LGTM

@SparkQA
Copy link

SparkQA commented Jun 2, 2016

Test build #59891 has finished for PR 13480 at commit 111bfaf.

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

@MLnick
Copy link
Contributor Author

MLnick commented Jun 2, 2016

Thanks, merged to master/branch-2.0

@asfgit asfgit closed this in ccd298e Jun 2, 2016
asfgit pushed a commit that referenced this pull request Jun 2, 2016
Clean up style for param setter methods in ALS to match standard style and the other setter in class (this is an artefact of one of my previous PRs that wasn't cleaned up).

## How was this patch tested?
Existing tests - no functionality change.

Author: Nick Pentreath <nickp@za.ibm.com>

Closes #13480 from MLnick/als-param-minor-cleanup.

(cherry picked from commit ccd298e)
Signed-off-by: Nick Pentreath <nickp@za.ibm.com>
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