Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[core] Fix checkstyle for RestWorkload #896

Merged
merged 1 commit into from
Jan 30, 2017
Merged

[core] Fix checkstyle for RestWorkload #896

merged 1 commit into from
Jan 30, 2017

Conversation

risdenk
Copy link
Collaborator

@risdenk risdenk commented Jan 18, 2017

Relates to #895

@risdenk risdenk self-assigned this Jan 18, 2017
@risdenk risdenk requested a review from busbey January 18, 2017 19:35
@risdenk
Copy link
Collaborator Author

risdenk commented Jan 18, 2017

@shivam-maharshi can you review?

@risdenk
Copy link
Collaborator Author

risdenk commented Jan 19, 2017

I checked this with: mvn clean validate | grep RestWorkload and mvn clean test

@risdenk risdenk modified the milestone: 0.13.0 Jan 19, 2017
@risdenk risdenk mentioned this pull request Jan 30, 2017
Closed
manolama
manolama approved these changes Jan 30, 2017
View reviewed changes
Copy link
Collaborator

@manolama manolama left a comment

Choose a reason for hiding this comment

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

LGTM

@risdenk risdenk merged commit 3e28ed6 into brianfrankcooper:master Jan 30, 2017
@risdenk risdenk deleted the checkstyle-RestWorkload branch January 30, 2017 22:16
@risdenk
Copy link
Collaborator Author

risdenk commented Jan 30, 2017

Thanks @manolama

tzm41 pushed a commit to tzm41/YCSB that referenced this pull request May 7, 2017
@risdenk @tzm41
[core] Fix checkstyle for RestWorkload (brianfrankcooper#896)
f8951fd
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manolama manolama manolama approved these changes

@busbey busbey Awaiting requested review from busbey

Assignees

@risdenk risdenk

Labels
improvement
Projects
None yet
Milestone
0.13.0
Development

Successfully merging this pull request may close these issues.
2 participants
@risdenk @manolama