Merge branch 'stable-2.15'

* stable-2.15:
  Update git submodules
  Use Logger's built-in string formatting where possible
  Doc: Fix code example in JS API
  Allow some sections of the change list to overflow
  Fix more comparisons of current user
  Revert "WorkQueue: Add metrics"
  Fix permissions checks on Gerrit API on current user
  GroupCacheImpl: Fix log message when UUID is not found
  Update git submodules
  Update git submodules
  WorkQueue.Executor#buildMetrics: Remove MetricMaker parameter
  ChangeQueryBuilder: Allow ownerin predicate to be evaluated by the index
  Update git submodules
  Update git submodules
  Update git submodules
  ldap.Helper: Use local logger and make logger in LdapRealm private
  Remove ValidationError#createLoggerSink to avoid passing around loggers
  LdapLoginServlet: Improve exception handling
  OperatingSystemMXBeanProvider: Log exception for ReflectiveOperationException
  HttpPluginServlet: Don't trim leading whitespace from about.md content
  ProjectConfig: Don't use JGit's StringUtils to convert to lower case
  Do not abort indexing if < 50% projects failed
  Revert "AllChangesIndexer: Don't abort when failing to open repository"
  WorkQueue: Don't fail when queue metric already exists
  WorkQueue: Sanitize metric name when queue is created
  DropWizardMetricMaker: Introduce method to sanitize metric name
  VersionedAccountDestinations: Remove unused createSink(String) method
  ProjectBasicAuthFilter: Add comment why cause is not logged
  BazelBuild: Fix exception message when command was interrupted
  GitwebServlet: Write only one log entry for CGI errors
  GitwebServlet: Log unexpected errors on error level
  PostGpgKeys: Remove unneeded use of Joiner
  Remove some logs for errors that are rethrown
  DropWizardMetricMaker: Improve error messages for invalid arguments
  DropWizardMetricMaker: Improve error message when metric name is invalid
  AllChangesIndexer: Don't abort when failing to open repository

Change-Id: I33e8a1c9498c6adecd4ffc7d8ef0cc78aeb28eea

java/com/google/gerrit/index/Schema.java

@@ -184,7 +184,7 @@ public class Schema<T> {
                 try {
                   v = f.get(obj);
                 } catch (OrmException e) {
-                  log.error(String.format("error getting field %s of %s", f.getName(), obj), e);
+                  log.error("error getting field {} of {}", f.getName(), obj, e);
                   return null;
                 }
                 if (v == null) {