uima-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tommaso Teofili (JIRA)" <...@uima.apache.org>
Subject [jira] Resolved: (UIMA-2043) SolrCASConsumers initialize method should be refactored
Date Mon, 07 Mar 2011 16:09:59 GMT

     [ https://issues.apache.org/jira/browse/UIMA-2043?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Tommaso Teofili resolved UIMA-2043.
-----------------------------------

    Resolution: Fixed

> SolrCASConsumers initialize method should be refactored
> -------------------------------------------------------
>
>                 Key: UIMA-2043
>                 URL: https://issues.apache.org/jira/browse/UIMA-2043
>             Project: UIMA
>          Issue Type: Improvement
>          Components: Sandbox
>            Reporter: Jörn Kottmann
>            Priority: Minor
>
> In the initialize method the following is done:
> String solrInstanceTypeParam = String.valueOf(context.getConfigParameterValue("solrInstanceType"));
> assert solrInstanceTypeParam != null;
> The assert will always be true since String.valueOf never returns a null reference, even
when
> you pass a null reference to it, it will return the string "null".
> getConfigParameterValue can actually return null, and we should fail the initialization
if
> not all necessary parameters to run it are specified in the descriptor. It should fail
> with a meaningful error message.
> Using assert is not a safe way to fail the initialization because asserts are usually
> disabled. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message