Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev 1.0.3 #1

Merged
merged 97 commits into from
Nov 13, 2021
Merged

Dev 1.0.3 #1

merged 97 commits into from
Nov 13, 2021

Conversation

weipengfei-sj
Copy link
Owner

What is the purpose of the change

(For example: EngineConn-Core defines the the abstractions and interfaces of the EngineConn core functions.
The Engine Service in Linkis 0.x is refactored, EngineConn will handle the engine connection and session management.
Related issues: apache#590. )

Brief change log

(for example:)

  • Define the core abstraction and interfaces of the EngineConn Factory;
  • Define the core abstraction and interfaces of Executor Manager.

Verifying this change

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

  • Added tests for submit and execute all kinds of jobs to go through and verify the lifecycles of different EngineConns.

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): (yes / no)
  • Anything that affects deployment: (yes / no / don't know)
  • The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: (yes / no)

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

lordk911 and others added 30 commits October 21, 2021 18:18
…d, resuslt in can't find instace from eureka. for example,hostname:linkis-cg-linkis-cg-engineconnmanager-1, aplicationName:linkis-cg-engineconnmanager,the service registed in eureka "linkis-cg-linkis-cg-engineconnmanager-1:linkis-cg-linkis-cg-engineconnmanager:9102", instanceId.toLowerCase.indexOf(applicationName.toLowerCase) will return 0.
reason: in 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'
for example,
hostname: linkis-cg-engineconnmanager-0.linkis-svc.default.svc.cluster.local
serviceName:linkis-cg-engineconnmanager
str:linkis-cg-engineconnmanager-linkis-cg-engineconnmanager-0.linkis-svc.default.svc.cluster.local:9104
after setStringValue:
instance = -0.linkis-svc.default.svc.cluster.local:9104
correct value should be: linkis-cg-engineconnmanager-0.linkis-svc.default.svc.cluster.local:9104
2. Add special judgment for uri without user login.
Adjust some unnecessary parameters in the BML APIs
…verLoader

bugFix: when hostname start with applicationName, will cause parse faile…
…CloumnWidth

bugFix:unified the width of all instance related field to be 128.
bugFix:instance Label parase failed when hostname contains servicename.
Add special judgment for uri without user login in Linkis-module.
iture123 and others added 27 commits November 10, 2021 22:06
remove old license and add new apache license for dev-1.0.3
Resolve the code conflicts
Add the support of EngineConn common operators and OnceEngineConn common operators.
Unify the exception class of engineConnPlugin in one package
[ISSUS#1080] bugfix:build dist ignore empty directory of 'python' after upgrade maven-assembly-plugin 2.3 to 3.2.0
remove some limit of package and some junk file
@weipengfei-sj weipengfei-sj merged commit 9a61f5e into weipengfei-sj:dev-1.0.3 Nov 13, 2021
weipengfei-sj pushed a commit that referenced this pull request Apr 19, 2022
weipengfei-sj pushed a commit that referenced this pull request Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.