You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@linkis.apache.org by GitBox <gi...@apache.org> on 2021/10/29 08:11:14 UTC

[GitHub] [incubator-linkis] lunescode opened a new pull request #1053: bugFix:unified the width of all instance related field to be 128.

lunescode opened a new pull request #1053:
URL: https://github.com/apache/incubator-linkis/pull/1053


   ### What is the purpose of the change
   Iin table linkis_cg_manager_service_instance and table linkis_cg_manager_service_instance_metrics, 'instance' is 128,but in linkis_ps_instance_label_relation(service_instance)、linkis_cg_manager_label_service_instance(service_instance)、linkis_cg_manager_engine_em(engine_instance、em_instance) is 64.
   if hostname use fqdn, then length will longer than 64,such as 'linkis-cg-engineconnmanager-0.linkis-svc.default.svc.cluster.local'
   
   ### Brief change log
   - modify linkis_ps_instance_label_relation(service_instance)  64->128 ;
   - modify linkis_cg_manager_label_service_instance(service_instance) 64->128;
   - modify linkis_cg_manager_engine_em(engine_instance、em_instance) 64->128; 
   
   ### Verifying this change
   This change is a trivial rework / code cleanup without any test coverage.  
   
   ### Does this pull request potentially affect one of the following parts:
   - Dependencies (does it add or upgrade a dependency): (no)
   - Anything that affects deployment: ( no )
   - The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: ( no)
   
   ### Documentation
   - Does this pull request introduce a new feature? (no)
   - If yes, how is the feature documented? (not applicable)


-- 
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: dev-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org


[GitHub] [incubator-linkis] peacewong merged pull request #1053: bugFix:unified the width of all instance related field to be 128.

Posted by GitBox <gi...@apache.org>.
peacewong merged pull request #1053:
URL: https://github.com/apache/incubator-linkis/pull/1053


   


-- 
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: dev-unsubscribe@linkis.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@linkis.apache.org
For additional commands, e-mail: dev-help@linkis.apache.org