Почему?

то есть в первом варианте я сначала добавляю созданный сервис, а потом создаю? 🙃 Но так-то Артем оказался прав, валидатору не понравилось именно второе обращение к кешу, Проверил и даже если его отдельно еще раз вызвать - сразу ошибка.

в первом варианте ты его создаешь и запихиваешь в кеш а потом оттуда извлекаешь и присваиваешь переменной. а валидатор скорее всего проверяет что ты присваиваешь именно созданный сервис, а не тот который из кеша. поэтому правильный, и при этом понятный вариант выглядит так:

Service s = cache.getService(jndiName);
        if (s == null) {
            InitialContext iс = new InitialContext();
            s = (Service) iс.lookup(jndiName);
            cache.addService(s);
        }
        return s;
    }

... Да нет, как раз могу, потому как в 4 строке он создан и сохранен в кеше. А в 5й переменной уже присваивается забранная из кеша ссылка. Что, в свою очередь, во первых соответствует пятому пункту "должен добавлять в кэш сервис, найденный в контексте и (затем!)возвращать его." что само по себе не так важно, но при этом позволяет написать так:

cache.addService((Service) iс.lookup(jndiName));
 s = cache.getService(jndiName);
if (s == null) throw new КакойНибудьException("сервис не добавился");

А в "правильном" варианте мы можем создавать сотни сервисов, вместо возвращения одного и того же и при этом не знать об этом. Что, вроде как, противоречит принципу "fail fast".