[GitHub] [hadoop] ZanderXu commented on pull request #4311: HDFS-13522: IPC changes to support observer reads through routers.

[GitBox <gi...@apache.org>]

ZanderXu commented on PR #4311:
URL: https://github.com/apache/hadoop/pull/4311#issuecomment-1155951845

   @simbadzina @omalley @melissayou I create a draft [PR](https://github.com/apache/hadoop/pull/4441), and have some different points, like:
   
   - Proxy always with AlignmentContext, like RouterGSIContext which contains a boolean flag, like enableObserverRead
   - Always update the lastSeenStateId from active
   - If enableObserverRead is false, updateRequestState will not set lastSeenStateId in RPCHeader
   - It's easily to dynamically reconfigure enableObserverRead.
   
   I'm hope that we can continue push this Feature forward. Please help me review it.
   
   @simbadzina If you want to continue work for this feature, feel free to take over it. Thanks~


-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org