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.

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

mds/MDCache: kill a comipler warning #11254

Merged
merged 1 commit into from Oct 4, 2016
Merged

Conversation

xiexingguo
Copy link
Member

/home/jenkins-build/build/workspace/ceph-pull-requests/src/mds/MDCache.cc: In member function ‘void MDCache::request_cleanup(MDRequestRef&)’:
/home/jenkins-build/build/workspace/ceph-pull-requests/src/mds/MDCache.cc:9090:8: warning: unused variable ‘was_replay’ [-Wunused-variable]
bool was_replay = mdr->client_request && mdr->client_request->is_replay();

Signed-off-by: xie xingguo xie.xingguo@zte.com.cn

/home/jenkins-build/build/workspace/ceph-pull-requests/src/mds/MDCache.cc: In member function ‘void MDCache::request_cleanup(MDRequestRef&)’:
/home/jenkins-build/build/workspace/ceph-pull-requests/src/mds/MDCache.cc:9090:8: warning: unused variable ‘was_replay’ [-Wunused-variable]
   bool was_replay = mdr->client_request && mdr->client_request->is_replay();

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo xiexingguo added cephfs Ceph File System cleanup labels Sep 29, 2016
@xiexingguo
Copy link
Member Author

@jcsp @gregsfortytwo

@ghost
Copy link

ghost commented Sep 29, 2016

jenkins test this please

@jcsp
Copy link
Contributor

jcsp commented Sep 29, 2016

retest this please

@gregsfortytwo
Copy link
Member

lgtm, this must have gotten missed when Zheng fixed the one-at-a-time replay loop stuff.

@ghost
Copy link

ghost commented Oct 1, 2016

https://jenkins.ceph.com/job/ceph-pull-requests/12056/ failed again jenkins test this please

@jcsp jcsp merged commit 0aab9e2 into ceph:master Oct 4, 2016
@xiexingguo xiexingguo deleted the xxg-wip-fs-0929 branch October 6, 2016 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cephfs Ceph File System cleanup
Projects
None yet
3 participants