[GitHub] chickenlj commented on issue #2468: Simplify registry data and add a new service data store seperated from registry #2030

[GitBox <gi...@apache.org>]

chickenlj commented on issue #2468: Simplify registry data and add a new service data store seperated from registry #2030
URL: https://github.com/apache/incubator-dubbo/pull/2468#issuecomment-419840479
 
 
   * ServiceStore can replace ServiceStoreService as the entrance in RegistryConfig/ServiceConfig.
   * Delete useless attributes in ServiceStoreConfig.
   * Enrich method signature information, generalized inclusion and realize methods in PojoUtils may help.
   * ServiceStore's HA strategy: if the server cannot be reached, ignore and try background. You can reference FailbackRegistry.
   * ZK nodes written by ServiceStore should be changed to persistent
   * OPS will need to retrieve the ServiceStore instance and I think you should provide the exact URL format used by ServiceStore.
   * To decide which keys need to be registered to Registry, we can make it configurable by adding an item in dubbo.properties.
   * Add a switch to control the URL registered to Registry: register the complete URL to the registry by default.
   * Is it necessary to keep cache files locally in AbstractServiceStore? I think it's too complex, just log the error information in case of write failing can be enough.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org