Я лично больше прощаю статические методы и классы, чем некоторые люди, поэтому я бы сказал, что это зависит. Является ли статический метод reentrant и потокобезопасным? Тогда это может быть хорошо. В противном случае это, вероятно, будет плохо масштабироваться.
Одна проблема, которую я вижу, - это контекст данных. При использовании статического метода вы, вероятно, не имеете точного контроля над продолжительностью жизни соединения с базой данных? Альтернативой может быть сделать его нестатическим классом и методом и инициализировать его в статическом конструкторе службы WCF. Таким образом, вы сможете легче изменить поведение позже, если потребуется.
На самом деле, статические конструкторы в сервисах WCF плохи, если у них есть шанс на сбой, например, если база данных находится в состоянии AWOL на секунду. Дело в том, что это позволит более детально контролировать сам сервис.