[jira] [Comment Edited] (UIMA-4059) Checking for incorrect key modifications

["Marshall Schor (JIRA)" <de...@uima.apache.org>]

    [ https://issues.apache.org/jira/browse/UIMA-4059?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14207166#comment-14207166 ] 

Marshall Schor edited comment on UIMA-4059 at 11/14/14 4:43 PM:
----------------------------------------------------------------

If this was enabled, it would be theoretically be possible to not throw an exception, but rather "make it work" by automatically removing the FS from the indices, making the modification, and then adding it back.  (This won't work if the modification were inside an iterator - the iterator throw a ConcurrentModificationException, which is why this is just theoretically possible).




was (Author: schor):
If this was enabled, it would be possible to not throw an exception, but rather "make it work" by automatically removing the FS from the indices, making the modification, and then adding it back. The downside of doing this would be potentially poor performance.  So if this was done, we would probably want to log (maybe not every, but at least some) of the occurrences.

> Checking for incorrect key modifications
> ----------------------------------------
>
>                 Key: UIMA-4059
>                 URL: https://issues.apache.org/jira/browse/UIMA-4059
>             Project: UIMA
>          Issue Type: Improvement
>          Components: Core Java Framework
>            Reporter: Marshall Schor
>            Assignee: Marshall Schor
>            Priority: Minor
>             Fix For: 2.7.0SDK
>
>
> Address the issue raised in Jira UIMA-4049, as follows: Add an optional check that checks, for every set of a Feature value, whether or not that Feature is used as a key in any Sort or Set index, and if so, if the Feature Structure is currently in any index in any View; if so, an exception is thrown.  
> This additional check is normally disabled, but can be enabled by specifying the JVM property {{-Duima.check_invalid_fs_updates}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)