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

rgw: optimize time skew check #19511

Merged
merged 1 commit into from Dec 19, 2017
Merged

rgw: optimize time skew check #19511

merged 1 commit into from Dec 19, 2017

Conversation

qrGitHub
Copy link

@qrGitHub qrGitHub commented Dec 14, 2017 •
edited

  • Move query string check out of function is_time_skew_ok;

rgw: optimize time skew check
2c97b0c 
* Move query string check out of function is_time_skew_ok;

Signed-off-by: Bingyin Zhang <zhangbingyin@cloudin.cn>
mattbenjamin
mattbenjamin approved these changes Dec 14, 2017
View reviewed changes
Copy link
Contributor

@mattbenjamin mattbenjamin left a comment

Choose a reason for hiding this comment

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

ha! ok

@qrGitHub qrGitHub mentioned this pull request Dec 14, 2017
Merged
rzarzynski
rzarzynski approved these changes Dec 14, 2017
View reviewed changes
Copy link
Contributor

@rzarzynski rzarzynski left a comment

Choose a reason for hiding this comment

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

LGTM.

@yuriw
Copy link
Contributor

yuriw commented Dec 18, 2017

wip-yuri2-testing-2017-12-18-1657

@yuriw yuriw merged commit 69ffdd4 into ceph:master Dec 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mattbenjamin mattbenjamin mattbenjamin approved these changes

@rzarzynski rzarzynski rzarzynski approved these changes

Assignees
No one assigned
Labels
cleanup needs-qa rgw wip-yuri2-testing
Projects
None yet
Milestone
No milestone
5 participants
@qrGitHub @yuriw @mattbenjamin @rzarzynski @cbodley