code: @FFDCIgnore({ NoSuchElementException.class })
      protected Map<String, String> getMpConfig(MpConfigProxyService service, ClassLoader cl, String propertyName,  Map<String, String> map) {
          try {
              String value = service.getConfigValue(cl, propertyName, String.class).trim();
              if (!value.isEmpty()) {
                  map.put(propertyName, value);
              } else {
                  if (TraceComponent.isAnyTracingEnabled() && tc.isDebugEnabled()) {
                      Tr.debug(tc, propertyName + " is empty. Ignore it.");
                  }
              }
          } catch (NoSuchElementException e) {
              if (TraceComponent.isAnyTracingEnabled() && tc.isDebugEnabled()) {
                  Tr.debug(tc, propertyName + " is not in mpConfig.");
              }
          }
          return map;
      }
docs: no null check other than cl. make sure that the caller sets non null objects.