[
https://issues.apache.org/jira/browse/LUCENE-8804?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vamshi Vijay Nakkirtha updated LUCENE-8804:
-------------------------------------------
Description:
Today FieldType attribute map can be modifiable even after freeze. For all
other properties of FieldType, we do "checkIfFrozen()" before making the update
to the property but for attribute map, we does not seem to make such check.
[https://github.com/apache/lucene-solr/blob/releases/lucene-solr/8.0.0/lucene/core/src/java/org/apache/lucene/document/FieldType.java#L363]
we may need to add check at the beginning of the function similar to other
properties setters.
If this seems to be valid, i can raise a PR for this.
was:
Today FieldType attribute map can be modified even after freeze. For all other
properties of FieldType, we do "checkIfFrozen()" before making the update to
the property but for attribute map, we does not seem to make such check.
[https://github.com/apache/lucene-solr/blob/releases/lucene-solr/8.0.0/lucene/core/src/java/org/apache/lucene/document/FieldType.java#L363]
we may need to add check at the beginning of the function similar to other
properties setters.
If this seems to be valid, i can raise a PR for this.
> FieldType attribute map should not be modifiable after freeze
> -------------------------------------------------------------
>
> Key: LUCENE-8804
> URL: https://issues.apache.org/jira/browse/LUCENE-8804
> Project: Lucene - Core
> Issue Type: Bug
> Components: core/index
> Affects Versions: 8.0
> Reporter: Vamshi Vijay Nakkirtha
> Priority: Minor
> Labels: features, patch
> Attachments: LUCENE-8804.patch
>
>
> Today FieldType attribute map can be modifiable even after freeze. For all
> other properties of FieldType, we do "checkIfFrozen()" before making the
> update to the property but for attribute map, we does not seem to make such
> check.
>
> [https://github.com/apache/lucene-solr/blob/releases/lucene-solr/8.0.0/lucene/core/src/java/org/apache/lucene/document/FieldType.java#L363]
> we may need to add check at the beginning of the function similar to other
> properties setters.
>
> If this seems to be valid, i can raise a PR for this.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org
