[GitHub] [hadoop] xkrogen commented on pull request #3976: HDFS-16452. msync RPC should send to acitve namenode directly

2022-04-12 Thread GitBox
xkrogen commented on PR #3976: URL: https://github.com/apache/hadoop/pull/3976#issuecomment-1097422287 Sorry for the delay in my response @hfutatzhanghb. I want to make sure I understand your proposal fully. I think what you are saying is this: Currently when using `ConfiguredF

[GitHub] [hadoop] xkrogen commented on pull request #3976: HDFS-16452. msync RPC should send to acitve namenode directly

2022-02-24 Thread GitBox
xkrogen commented on pull request #3976: URL: https://github.com/apache/hadoop/pull/3976#issuecomment-1050121895 I agree with you that the current logic has RPCs that are theoretically unnecessary. I would be very open to some approach that allows `ObservedReadProxyProvider` to share infor

[GitHub] [hadoop] xkrogen commented on pull request #3976: HDFS-16452. msync RPC should send to acitve namenode directly

2022-02-18 Thread GitBox
xkrogen commented on pull request #3976: URL: https://github.com/apache/hadoop/pull/3976#issuecomment-1044778317 Quick disclaimer, it has been a while since I have looked at any proxy-provider code. This PR looks in the wrong direction to me. As you mentioned, `failoverProxy` is use