You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
### What changes are proposed in this pull request?
Fix string format specifier `%d` in `Preconditions.checkArgument` error
message template.
### Why are the changes needed?
any other format specifiers like `%d` are not recognized at the method
of `Preconditions.checkArgument`, only `%s` is accepted.
### Does this PR introduce any user facing changes?
none
Fixed#642Alluxio/Community#642
pr-link: #16360
change-id: cid-673b8647502d63d96d88ed59d3bd6829acf7d46d
jja725
pushed a commit
to jja725/alluxio
that referenced
this issue
Jan 27, 2023
### What changes are proposed in this pull request?
Fix string format specifier `%d` in `Preconditions.checkArgument` error
message template.
### Why are the changes needed?
any other format specifiers like `%d` are not recognized at the method
of `Preconditions.checkArgument`, only `%s` is accepted.
### Does this PR introduce any user facing changes?
none
FixedAlluxio#642Alluxio/Community#642
pr-link: Alluxio#16360
change-id: cid-673b8647502d63d96d88ed59d3bd6829acf7d46d
Preconditions.checkArgument
and friends take an optional message template that can have format specifier%s
only. Any other format specifiers like%d
are not recognized and output as is. See guava doc: https://guava.dev/releases/31.1-jre/api/docs/com/google/common/base/Preconditions.htmlThere are several places across the code base that are using
%d
etc., in the error message template, e.g.https://github.com/Alluxio/alluxio/blob/6982d6c759e7180c9b1ae00ea7e6c5229765ff2e/core/server/worker/src/main/java/alluxio/worker/block/UfsInputStreamCache.java#L278
Find across the code base such occurrences and fix them.
The text was updated successfully, but these errors were encountered: