Mikhail Khludnev
added a comment - 25/Nov/15 12:16 patch seems fine, but the last chunk scares me much. as far as I get, nobody use it, it should be a getter like
public CharSequence[] getMultiFields() {
return getQueryConfigHandler().get(ConfigurationKeys.MULTI_FIELDS);
}
formally it's non-compatible change, but it think it's worth to break it even in 5.4, as an alternative let's fix it in trunk only.

Thanks Mikhail. I have created LUCENE-6911 to deal with StandardQueryParser.getMultiFields separately for clarity. Perhaps for 5.4 the broken method could be left as-is and marked as deprecated and in trunk we remove the broken method?

Christine Poerschke
added a comment - 25/Nov/15 14:44 Thanks Mikhail. I have created LUCENE-6911 to deal with StandardQueryParser.getMultiFields separately for clarity. Perhaps for 5.4 the broken method could be left as-is and marked as deprecated and in trunk we remove the broken method?