nifi-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From olegz <...@git.apache.org>
Subject [GitHub] nifi pull request: NIFI-1192 added support for Dynamic Properties ...
Date Fri, 20 Nov 2015 22:21:04 GMT
Github user olegz commented on the pull request:

    https://github.com/apache/nifi/pull/129#issuecomment-158542457
  
    @trkurc @joewitt @markap14 
    Guys, please see the updated commit. This one has a bit more work as it includes some
polishing since we discovered few more issues, one being that there was an non-existing property
etc. See commit message for more details.
    
    Tony, in relation to your last message, they will not override any previously set property
since NiFi will not allow it. Basically one of the biggest changes that went in with this
commit is setting _name_ and _displayName_. So when one attempts to set the property (e.g.,
'client.id') as dynamic property, the NiFi will pop up with the message 'A property with this
name already exists.’. So I think we're good, but let me know if you still see an issue.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message