bbende commented on issue #3546: NIFI-6025: Include Processor 'scheduled state' 
(i.e., Enabled or Disa…
URL: https://github.com/apache/nifi/pull/3546#issuecomment-511010954
 
 
   @markap14 it seems like the scheduled state only gets applied if some other 
change is also made to the component which triggers the update.
   
   Example:
   
   - Created a PG with proc1 enabled, proc2 disabled
   - Start version control
   - Import another instance of it and verify the procs have correct state
   - Change original PG so that proc1 is disabled, and proc2 is enabled, also 
moved position of proc2 to make a local change
   - Commit version 2
   - Change version on second PG and it correctly updates proc2, but leaves 
proc1 enabled
   
   When stepping through in the debugger it never enters the updateProcessor 
method for proc1. The only reason I can think is because it wasn't considered 
an affected component b/c nothing in the flow diff showed proc1.
   
   This probably ties back into Matt's question because if we start adding 
schedule state into the flow diff then it will become a local change by 
default, and then we have to decide if we want to add the code to filter it out 
on NiFi side.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to