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.

Sign up for GitHub

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

Jump to bottom

[fix]Workspace management user management: query sql exception repair #967

Merged
merged 1 commit into from
Oct 17, 2022
Merged

[fix]Workspace management user management: query sql exception repair #967

merged 1 commit into from
Oct 17, 2022

Conversation

ichenfeiyang
Copy link

What is the purpose of the change

Workspace Management - User Management Module，Query SQL report an error，I tried to fix it.
image

Brief change log

I changed this file :com.webank.wedatasphere.dss.framework.workspace.dao.DSSWorkspaceUserMapper
Line 62
image

Verifying this change

image

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

  • Dependencies (does it add or upgrade a dependency): (yes / no) - no
  • Anything that affects deployment: (yes / no / don't know) - no
  • The Core framework, i.e., AppConn, Orchestrator, ApiService.: (yes / no) - no

Documentation

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

jacktao007
jacktao007 approved these changes Oct 17, 2022
View reviewed changes
Copy link

@jacktao007 jacktao007 left a comment

Choose a reason for hiding this comment

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

LGTM.

@ichenfeiyang ichenfeiyang changed the title 【dss-framework-project-server】工作空间管理-用户管理：查询sql异常修复 [fix]Workspace management user management: query sql exception repair Oct 17, 2022
zqburde
zqburde approved these changes Oct 17, 2022
View reviewed changes
Copy link
Contributor

@zqburde zqburde left a comment •
edited
Loading

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 eed61d7 into WeBankFinTech:dev-1.1.0 Oct 17, 2022
@zqburde zqburde added the bug Something isn't working label Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jacktao007 jacktao007 jacktao007 approved these changes

@zqburde zqburde zqburde approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ichenfeiyang @jacktao007 @zqburde