[LUCENE-9303] There may be can simpler in DefaultIndexingChain

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 8.2
• Component/s: core/index
• Labels: newdev
• Lucene Fields: New
• Assignee: Unassigned
• Reporter: kkewwei
• Created: 02/Apr/20 13:46
• Updated: 28/Aug/22 16:01

Description

 In  DefaultIndexingChain.processField()

if (fieldType.stored()) {
  if (fp == null) {
    fp = getOrAddField(fieldName, fieldType, false);
  }
  if (fieldType.stored()) {
    String value = field.stringValue(); 
    ......
    try { 
      storedFieldsConsumer.writeField(fp.fieldInfo, field);
    } catch (Throwable th) {
      ......
    }
  }
}

 If there has need to add the second if, because fieldType.stored() is executed before.

 

Issue Links

• links to: GitHub Pull Request #993