useDocValuesAsStored on unique key field

[Bram Van Dam <br...@intix.eu>]

Howdy,

I'm getting a lot of warnings in the logs about unique key not being 
stored when using stored=false and useDocValuesAsStored=true.

An old post on the mailing list suggests that this is probably (?) not a 
problem:

https://lists.apache.org/thread/kmt1thg70fffo0dg29r35rc9gpyppp9h

However, the check is still unchanged on master:

IndexSchema.java:

if (!uniqueKeyField.stored()) {
   log.warn("{} is not stored ...

If it is indeed safe to use, shouldn't this check be changed to
!uniqueKeyField.stored() && !uniqueKeyField.useDocValuesAsStored()

And if it isn't safe to use, that would be nice to know as well ..

Thanks,

  - Bram