Skip to content

Conversation

zhangconan
Copy link
Contributor

Add a new feature to support opengauss "on duplicate key update nothing" syntax. I have executed mvn clean package locally, and all code format checks and unit tests have passed.

Copy link
Contributor

@manticore-projects manticore-projects left a comment

Choose a reason for hiding this comment

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

Looks good in general, but please avoid Wildcard imports and import single Classes instead.

import net.sf.jsqlparser.statement.ReturningClause;
import net.sf.jsqlparser.statement.Statement;
import net.sf.jsqlparser.statement.StatementVisitor;
import net.sf.jsqlparser.statement.select.PlainSelect;
Copy link
Contributor

Choose a reason for hiding this comment

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

Please avoid wildcard imports.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK. I will adjust the code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have adjusted.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':publishMavenJavaPublicationToOssrhRepository'.
Failed to publish publication 'mavenJava' to repository 'ossrh'
Could not PUT 'https://central.sonatype.com/repository/maven-snapshots/com/github/jsqlparser/jsqlparser/5.4-SNAPSHOT/jsqlparser-5.4-20250828.114640-16.jar'. Received status code 401 from server: Unauthorized

There seems to be a problem with the username or password of the Maven repository

@zhangconan
Copy link
Contributor Author

FAILURE: Build failed with an exception.

  • What went wrong:
    Execution failed for task ':publishMavenJavaPublicationToOssrhRepository'.

Failed to publish publication 'mavenJava' to repository 'ossrh'
Could not PUT 'https://central.sonatype.com/repository/maven-snapshots/com/github/jsqlparser/jsqlparser/5.4-SNAPSHOT/jsqlparser-5.4-20250828.114640-16.jar'. Received status code 401 from server: Unauthorized

There seems to be a problem with the username or password of the Maven repository

@manticore-projects manticore-projects merged commit 468aefa into JSQLParser:master Aug 28, 2025
2 of 3 checks passed
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.

2 participants