Speaking of consistency there are heaps of cases in our code base where we check against _null_ and throw IllegalArgumentException. So I think it would be more important to align with the rest of the Search code base. Unless I see to fail why this null check is any different as the others.