lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vamshi Vijay Nakkirtha (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (LUCENE-8804) FieldType attribute map should not be modifiable after freeze
Date Sat, 18 May 2019 08:27:00 GMT

     [ 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-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message