Alanxtl opened a new issue, #3243:
URL: https://github.com/apache/dubbo-go/issues/3243

   ### Description
   
   The API / behavior was originally removed during the changes in:
   - apache/dubbo-go#3046
   
   Later, we found this caused downstream compatibility problems for Dubbo-Go:
   - apache/dubbo-go#3070
   
   In particular, removing the `grpc.WithBlock()`-related behavior in `gost` 
affected the initialization semantics along this dependency chain:
   
   `dubbo-go` -> `gost` -> `etcd/clientv3` -> `grpc`
   
   As discussed in #3070, this broke the expected behavior in the etcd registry 
path and caused unit test failures in Dubbo-Go.
   
   Now the upstream etcd side has already updated the related implementation / 
discussion:
   - etcd-io/etcd#20910
   - https://github.com/etcd-io/etcd/issues/20910#issuecomment-4048775213
   
   ***So I think it is time to add this API / behavior back in `gost`.***
   
   ### Motivation
   
   The original removal was understandable at that time, but it introduced real 
downstream compatibility issues.
   
   Since the upstream dependency has now moved forward, restoring this API / 
behavior would help downstream projects such as Dubbo-Go remove temporary 
compatibility workarounds and recover the expected client initialization 
semantics.
   
   ### References
   
   - Original removal: apache/dubbo-go#3046
   - Compatibility regression discussion: apache/dubbo-go#3070
   - Upstream etcd update: etcd-io/etcd#20910


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