You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@griffin.apache.org by "Nikolay Sokolov (JIRA)" <ji...@apache.org> on 2019/02/20 15:52:00 UTC
[jira] [Commented] (GRIFFIN-232) Implement pluggable Predicators in
PredicatorFactory
[ https://issues.apache.org/jira/browse/GRIFFIN-232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16773125#comment-16773125 ]
Nikolay Sokolov commented on GRIFFIN-232:
-----------------------------------------
[~aborgatin] I think we can keep default ones hardcoded, or at least provide default value for "predicates" property, to keep backward compatibility with older versions. We don't want to break existing deployments because of missing property.
> Implement pluggable Predicators in PredicatorFactory
> ----------------------------------------------------
>
> Key: GRIFFIN-232
> URL: https://issues.apache.org/jira/browse/GRIFFIN-232
> Project: Griffin (Incubating)
> Issue Type: Improvement
> Reporter: Aleksandr Borgatin
> Priority: Minor
>
> I propose
> # add property predicates to application.properties
> {code:java}
> predicates={'file.exist':'org.apache.griffin.core.job.FileExistPredicator'}{code}
> # change PredicatorFactory for reading values from property predicates, loading class and creating instance of Predicate
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)