HBASE-24163 MOB compactor implementations should use format specifiers when calling String.format (#1479)

Signed-off-by: stack <stack@apache.org>
Signed-off-by: Viraj Jasani <vjasani@apache.org>
Signed-off-by: Josh Elser <elserj@apache.org>
This commit is contained in:
Sean Busbey 2020-04-10 11:00:53 -05:00 committed by GitHub
parent ac4e94e868
commit 09d20fb4d7
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 4 deletions

View File

@ -435,7 +435,7 @@ public class DefaultMobStoreCompactor extends DefaultCompactor {
} else {
throw new IOException(String.format("MOB cell did not contain a tablename " +
"tag. should not be possible. see ref guide on mob troubleshooting. " +
"store={} cell={}", getStoreInfo(), c));
"store=%s cell=%s", getStoreInfo(), c));
}
}
}
@ -491,7 +491,7 @@ public class DefaultMobStoreCompactor extends DefaultCompactor {
} else {
throw new IOException(String.format("MOB cell did not contain a tablename " +
"tag. should not be possible. see ref guide on mob troubleshooting. " +
"store={} cell={}", getStoreInfo(), c));
"store=%s cell=%s", getStoreInfo(), c));
}
} else {
String errMsg = String.format("Corrupted MOB reference: %s", c.toString());

View File

@ -246,7 +246,7 @@ public class FaultyMobStoreCompactor extends DefaultMobStoreCompactor {
} else {
throw new IOException(String.format("MOB cell did not contain a tablename " +
"tag. should not be possible. see ref guide on mob troubleshooting. " +
"store={} cell={}", getStoreInfo(), c));
"store=%s cell=%s", getStoreInfo(), c));
}
} else {
// If the value is not larger than the threshold, it's not regarded a mob. Retrieve
@ -270,7 +270,7 @@ public class FaultyMobStoreCompactor extends DefaultMobStoreCompactor {
} else {
throw new IOException(String.format("MOB cell did not contain a tablename " +
"tag. should not be possible. see ref guide on mob troubleshooting. " +
"store={} cell={}", getStoreInfo(), c));
"store=%s cell=%s", getStoreInfo(), c));
}
}
}