[ 
https://issues.apache.org/jira/browse/HDFS-17124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17747202#comment-17747202
 ] 

Simbarashe Dzinamarira edited comment on HDFS-17124 at 8/1/23 5:46 PM:
-----------------------------------------------------------------------

"dfs.client.rbf.observer.enable" was not an elegant way to support observer 
reads with the IPFailoverProxyProvider. I think we should not spread this 
config to RouterObserverReadProxy provide, but instead deprecate it.


was (Author: simbadzina):
"dfs.client.rbf.observer.enable" was a hack-ish way to support observer reads 
with the IPFailoverProxyProvider. I think we should not spread with config to 
RouterObserverReadProxy provide, but instead deprecate it.

> RBF: RouterObserverReadProxyProvider should respect observer read opt-out via 
> dfs.client.rbf.observer.enable flag.
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: HDFS-17124
>                 URL: https://issues.apache.org/jira/browse/HDFS-17124
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Simbarashe Dzinamarira
>            Assignee: Simbarashe Dzinamarira
>            Priority: Minor
>
> The "dfs.client.rbf.observer.enable" flag was add to allow a client using 
> IPFailoverProxyProvider to use observer reads with routers.
> Now we have implemented the RouterObserverReadProxyProvider which better 
> supported observer reads, i.e performing msync automatically.
> In our environment, we still have clients that opted out of observer reads by 
> setting "dfs.client.rbf.observer.enable
> =false". The RouterObserverReadProxyProvider should respect the opt-out 
> decision of these clients.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to