[ https://issues.apache.org/jira/browse/SOLR-6013?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13986460#comment-13986460 ]
ASF subversion and git services commented on SOLR-6013: ------------------------------------------------------- Commit 1591574 from sha...@apache.org in branch 'dev/branches/branch_4x' [ https://svn.apache.org/r1591574 ] SOLR-6013: Fix method visibility of Evaluator, refactor DateFormatEvaluator for extensibility > Fix method visibility of Evaluator, refactor DateFormatEvaluator for > extensibility > ---------------------------------------------------------------------------------- > > Key: SOLR-6013 > URL: https://issues.apache.org/jira/browse/SOLR-6013 > Project: Solr > Issue Type: Improvement > Components: contrib - DataImportHandler > Affects Versions: 4.7 > Reporter: Aaron LaBella > Assignee: Shalin Shekhar Mangar > Fix For: 4.9, 5.0 > > Attachments: 0001-add-getters-for-datemathparser.patch, > 0001-change-method-access-to-protected.patch, > 0001-change-method-variable-visibility-and-refactor-for-extensibility.patch, > SOLR-6013.patch > > Original Estimate: 1h > Remaining Estimate: 1h > > This is similar to issue 5981, the Evaluator class is declared as abstract, > yet the parseParams method is package private? Surely this is an oversight, > as I wouldn't expect everyone writing their own evaluators to have to deal > with parsing the parameters. > Similarly, I needed to refactor DateFormatEvaluator because I need to do some > custom date math/parsing and it wasn't written in a way that I can extend it. > Please review/apply my attached patch to the next version of Solr, ie: 4.8 or > 4.9 if I must wait. > Thanks! -- This message was sent by Atlassian JIRA (v6.2#6252) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org