Skip to content

HDDS-9213. Findbugs error in TestOMRatisSnapshots.#5221

Merged
GeorgeJahad merged 1 commit intoapache:masterfrom
GeorgeJahad:HDDS-9213
Aug 24, 2023
Merged

HDDS-9213. Findbugs error in TestOMRatisSnapshots.#5221
GeorgeJahad merged 1 commit intoapache:masterfrom
GeorgeJahad:HDDS-9213

Conversation

@GeorgeJahad
Copy link
Contributor

What changes were proposed in this pull request?

I accidentally merged this findbugs error into master:

M D DLS: Dead store to numberOfSstFiles in org.apache.hadoop.ozone.om.TestOMRatisSnapshots.testSnapshotBackgroundServices() At TestOMRatisSnapshots.java:[line 1245]

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-9213

How was this patch tested?

findbugs

@GeorgeJahad GeorgeJahad changed the title HDDS-9213. Find bugs error in TestOMRatisSnapshots. HDDS-9213. Findbugs error in TestOMRatisSnapshots. Aug 24, 2023
@GeorgeJahad GeorgeJahad merged commit 3126e96 into apache:master Aug 24, 2023
@smengcl
Copy link
Contributor

smengcl commented Aug 24, 2023

Hi @GeorgeJahad , thanks for the patch.

Note when there is only one commit in the PR, Github merge will use that commit message as the merged commit's message by default. Please watch out for the commit message when merging a PR next time.

@GeorgeJahad
Copy link
Contributor Author

Hi @GeorgeJahad , thanks for the patch.

Note when there is only one commit in the PR, Github merge will use that commit message as the merged commit's message by default. Please watch out for the commit message when merging a PR next time.

Will do. thanks!

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.

2 participants