STRRL commented on issue #6939:
URL: https://github.com/apache/opendal/issues/6939#issuecomment-3647985060

   Hi Xunao, I'm working on issue #6939 (split logging layer), but I've 
discovered a dependency issue:
   
     The retry layer tests in core/core/src/layers/retry.rs use LoggingLayer 
for debugging. If I split logging first, there's a circular dependency problem:
     - opendal-layer-logging depends on opendal-core
     - opendal-core tests can't use opendal-layer-logging as dev-dependency 
(Cargo sees two different crate instances)
   
     Proposed approach:
     1. Split retry layer first → creates opendal-layer-retry
     2. Split logging layer second → creates opendal-layer-logging
     3. opendal-layer-retry can then use opendal-layer-logging as a 
dev-dependency for its tests
   
    
   > This message is also composed by AI. 
   
   > BTW @zhiqiang-zz is also me, sorry for the messing 🤪
   
   
   I see somebody already started working on 
https://github.com/apache/opendal/issues/6947, I am not sure it's OK that let 
me take over.


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

Reply via email to