zpinto opened a new pull request, #2890:
URL: https://github.com/apache/helix/pull/2890

   ### Issues
   
   - [x] Implementation to allow gateway participant to only send transitions 
to the targetState to the client application.
   
   ### Description
   
   The intended behavior for HelixGateway is to only send the targetState to 
the client application. A major refactor of Helix internals will be required to 
achieve this, as the controller will send a state transition message for each 
state in the state machine instead of just the final target state.
   
   Until that refactor, the gateway participant will consult the IdealState of 
the resource for the pending state transition and forward a shard assignment 
message that has a the to state from the IdealState instead. All following ST 
messages from the controller will be auto-completed and NOT forwarded to the 
participant, as the client is already in the target state.
   
   ### Tests
   
   - [x] TestHelixGatewayParticipant.testOnlySendTargetState
   
   
   ### Changes that Break Backward Compatibility (Optional)
   
   NA
   
   ### Documentation (Optional)
   
   NA
   
   ### Commits
   
   - My commits all reference appropriate Apache Helix GitHub issues in their 
subject lines. In addition, my commits follow the guidelines from "[How to 
write a good git commit message](http://chris.beams.io/posts/git-commit/)":
     1. Subject is separated from body by a blank line
     1. Subject is limited to 50 characters (not including Jira issue reference)
     1. Subject does not end with a period
     1. Subject uses the imperative mood ("add", not "adding")
     1. Body wraps at 72 characters
     1. Body explains "what" and "why", not "how"
   
   ### Code Quality
   
   - My diff has been formatted using helix-style.xml 
   (helix-style-intellij.xml if IntelliJ IDE is used)
   


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to