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

bug: delete namespce will cause error #5912

Closed
1 task done
liyabing12138 opened this issue Jan 22, 2025 · 16 comments
Closed
1 task done

bug: delete namespce will cause error #5912

liyabing12138 opened this issue Jan 22, 2025 · 16 comments
Labels
type: bug Something isn't working

Comments

@liyabing12138
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Error querying database. Cause: java.sql.SQLSyntaxErrorException: Unknown column 'namespace_id' in 'where clause' ### The error may exist in class path resource [mappers/auth-path-sqlmap.xml] ### The error may involve org.apache.shenyu.admin.mapper.AuthPathMapper.findByNamespaceIds-Inline ### The error occurred while setting parameters ### SQL: SELECT id, auth_id, app_name, path, enabled, date_created, date_updated FROM auth_path WHERE namespace_id IN ( ? ) ### Cause: java.sql.SQLSyntaxErrorException: Unknown column 'namespace_id' in 'where clause' ; bad SQL grammar []

Expected Behavior

No response

Steps To Reproduce

No response

Environment

ShenYu version(s):

Debug logs

No response

Anything else?

No response
@liyabing12138 liyabing12138 added the type: bug Something isn't working label Jan 22, 2025
@xcsnx
Copy link
Member

xcsnx commented Jan 23, 2025

Have you executed the SQL script for version update?

@Aias00 Aias00 mentioned this issue Jan 23, 2025
Closed
3 tasks
@Aias00 Aias00 changed the title 删除名称空间报错问题 bug: delete namespce will cause error Jan 23, 2025
@Aias00
Copy link
Contributor

Aias00 commented Jan 23, 2025

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

The SQL has been validated and the code format needs to be adjusted again

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

2.7.0-upgrade-2.7.1-mysql.sql

@xcsnx
Copy link
Member

xcsnx commented Jan 23, 2025

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

@Aias00
Copy link
Contributor

Aias00 commented Jan 23, 2025

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

he submit a pr already

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

@xcsnx
Copy link
Member

xcsnx commented Jan 23, 2025

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.

Image

This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.

Image

This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

@xcsnx xcsnx assigned xcsnx and unassigned xcsnx Jan 23, 2025
@Aias00
Copy link
Contributor

Aias00 commented Jan 23, 2025

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

I submit a new pr

@Aias00
Copy link
Contributor

Aias00 commented Jan 23, 2025

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

I submit a new pr

hi, did not see that? link pls?

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

I submit a new pr

hi, did not see that? link pls?

@liyabing12138
Copy link
Contributor Author

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

I submit a new pr

hi, did not see that? link pls?

2e48056

Is that what I mean? I don't know if I understand correctly.

@Aias00
Copy link
Contributor

Aias00 commented Jan 23, 2025

Have you executed the SQL script for version update?

Have you executed the SQL script for version update?

hi. the table name in error info is "auth_path", which is not in upgrade sql

Okay, I will fix this error

I adjusted the code compilation issue

I'm sorry, the modification of this PR (#5913) is incorrect.
Image
This should be querying appAuthMapper instead of authPathMapper. Would you be willing to change this PR to fix this bug

Continue to solve it

close the previous pr, and submit a new pr to fix this

ok

I submit a new pr

hi, did not see that? link pls?

2e48056

Is that what I mean? I don't know if I understand correctly.

pls add my wechat: aias00

@Aias00 Aias00 reopened this Jan 23, 2025
@xcsnx xcsnx mentioned this issue Jan 23, 2025
Merged
3 tasks
@xcsnx xcsnx closed this as completed Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Aias00 @liyabing12138 @xcsnx