dpol1 commented on PR #3011:
URL: https://github.com/apache/hugegraph/pull/3011#issuecomment-4380286897

   Thanks @imbajin. Addressed all three points.
   
     **1. Reconnect lifecycle**                                                 
                                                                                
           
      
     Confirmed: no `MetaManager`/`MetaDriver` reconnect callback exists today. 
`resetMetaListenerForReconnect()` is a manual hook only — clarified in          
            
     `CachedSchemaTransactionV2` comment + Javadoc, TODO kept for wiring once a 
reconnect callback is exposed. This PR is not an automatic reconnect-recovery 
mechanism.
                                                                                
                                                                                
           
     **2. `~create_time` round-trip**
   
     Out of scope, filed as follow-up: #3013 .
                                                                                
                                                                                
           
     **3. Compatibility note**
                                                                                
                                                                                
           
     Added to PR description and as code comment at 
`MetaManager.SchemaCacheClearEvent.fromValue()` (legacy plain-string branch):
                                                                                
                                                                                
           
     > This PR does not change persisted schema data. It only changes the 
runtime schema-cache-clear meta-event value so new events can carry a per-JVM 
`source` id. New 
     consumers still accept legacy plain-string graph names, so mixed-version 
rollouts are safe.                                                              
             
                                
     `MetaManagerSchemaCacheClearEventTest` already covers both formats (legacy 
plain-string + JSON with `graph`/`source`). Before/after diagram also in PR 
description.   
                                                                                
                                                                                
           


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