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

Linkis1.2.0[datamodel][dataassets][datawarehouse] #952

Merged
merged 7 commits into from
Dec 30, 2022

Conversation

Zosimer
Copy link

@Zosimer Zosimer commented Sep 14, 2022

Linkis1.2.0+dss1.2.0[datamodel][dataassets][datawarehouse],Realize data model management, connect with Atlas, and realize blood relationship management

What is the purpose of the change

(For example: AppConn Core defines the abstractions and interfaces of the AppConn core functions.
It is the core concept that enables DSS to easily and quickly integrate various upper-layer web systems.
Related issues: #953. )

Linkis1.2.0+dss1.2.0[datamodel][dataassets][datawarehouse],Realize data model management, connect with Atlas, and realize blood relationship management
@Zosimer Zosimer changed the title dataassets dataWarehouse dataModel Linkis1.2.0+dss1.2.0[datamodel][dataassets][datawarehouse] Sep 14, 2022
@Zosimer Zosimer changed the title Linkis1.2.0+dss1.2.0[datamodel][dataassets][datawarehouse] Linkis1.2.0[datamodel][dataassets][datawarehouse] Sep 14, 2022
Linkis1.2.0+dss1.2.0[datamodel][dataassets][datawarehouse],Realize data model management, connect with Atlas, and realize blood relationship management
Copy link

@dlimeng dlimeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many of them are the source files of my original version. Check a few key points and suggest modifications.


wds.workspace.client.dws.version=v1

wds.wedatasphere.data.assert.client.strategy=static
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

recommend wds.wedatasphere.data.assert.client.strategy=token




wds.wedatasphere.data.model.client.strategy=static
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

token Way

getBaseMapper().insert(newOne);

//同步atlas
CreateModelTypeResult result = assertsSyncService.syncCreateModel(
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

publisher.publishEvent(new CreateModelEvent(this
, DataModelSecurityContextHolder.getContext().getDataModelAuthentication().getUser()
, vo.getFieldIdentifier()
, ClassificationConstant.DIMENSION));

getBaseMapper().update(updateOne, Wrappers.<DssDatamodelDimension>lambdaUpdate().eq(DssDatamodelDimension::getId,id));

//同步atlas
UpdateModelTypeResult updateModelTypeResult = assertsSyncService.syncUpdateModel(
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

publisher.publishEvent(new UpdateModelEvent(this
,DataModelSecurityContextHolder.getContext().getDataModelAuthentication().getUser()
,vo.getFieldIdentifier()
,orgFieldIdentifier
,ClassificationConstant.DIMENSION));

}
getBaseMapper().deleteById(id);
//同步资产
assertsSyncService.syncDeleteModel(new DeleteModelEvent(this
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

publisher.publishEvent(new DeleteModelEvent(this
,DataModelSecurityContextHolder.getContext().getDataModelAuthentication().getUser()
,dssDatamodelDimension.getFieldIdentifier()
,ClassificationConstant.DIMENSION));

PreconditionUtil.checkState(1 == insert, DwException.stateReject("create dw layer failed"));

// 建立关联
try {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asynchronous association


// 删除关联
try {
LinkisDataAssetsRemoteClient dataAssetsRemoteClient = LinkisRemoteClientHolder.getDataAssetsRemoteClient();
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asynchronous association

PreconditionUtil.checkState(1 == i, DwException.stateReject("update layer failed"));

// 更新关联
if (!Objects.equals(orgEnName, layer.getEnName())) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asynchronous association


// 建立关联
try {
LinkisDataAssetsRemoteClient dataAssetsRemoteClient = LinkisRemoteClientHolder.getDataAssetsRemoteClient();
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asynchronous association

int i = this.dwThemeDomainMapper.deleteById(record);
PreconditionUtil.checkState(1 == i, DwException.stateReject("remove action failed"));

// 删除关联
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Asynchronous association

int versionCount = (int) preReferences.stream().filter(e->{
DssDatamodelTableColumns temp = gson.fromJson(e.getTableParams(),DssDatamodelTableColumns.class);
//名称相同且类型相同
return modeType.getType() == temp.getModelType()&&StringUtils.equals(name,temp.getModelName());
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

temp.getModelType() !=null && modeType.getType() == temp.getModelType() && StringUtils.equals(name, temp.getModelName());

Copy link
Contributor

@zqburde zqburde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zqburde zqburde merged commit 8469876 into WeBankFinTech:dev-1.2.0-atlas Dec 30, 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.

3 participants