GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — PostgreSQL INSERT ... ON CONFLICT Issue #1551 by manticore-projects · Pull Request #1552 · JSQLParser/JSqlParser

Conversation

@manticore-projects

@manticore-projects manticore-projects commented Jun 3, 2022

Copy link
Copy Markdown
Contributor

Fixes #1551
Supports PostgreSQL INSERT ... ON CONFLICT ... DO ... according to https://www.postgresql.org/docs/current/sql-insert.html
Allow keyword KEEP, which enables another Special Oracle Test (191 are passing now)

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
manticore-projects added 22 commits November 22, 2021 14:09
@manticore-projects
Adjust Gradle to JUnit 5
de805ab 
Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10
@manticore-projects
Do not mark SpeedTest for concurrent execution
d5a6dca
@manticore-projects
Remove unused imports
a9d0503
@manticore-projects
Adjust Gradle to JUnit 5
6dfa05f 
Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10
@manticore-projects
Do not mark SpeedTest for concurrent execution
8f0bfe6
@manticore-projects
Remove unused imports
5cd0974
@manticore-projects
Merge remote-tracking branch 'origin/master'
5f11d3f
@manticore-projects
Merge https://github.com/JSQLParser/JSqlParser
3ab04b0
@manticore-projects
Merge branch 'JSQLParser:master' into master
de94651
@manticore-projects
Adjust Gradle to JUnit 5
c76ae00 
Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10
@manticore-projects
Do not mark SpeedTest for concurrent execution
b4d111e
@manticore-projects
Remove unused imports
0a01f32
@manticore-projects
Adjust Gradle to JUnit 5
f55ab13 
Parallel Test execution
Gradle Caching
Explicitly request for latest JavaCC 7.0.10
@manticore-projects
Do not mark SpeedTest for concurrent execution
884583d
@manticore-projects
Remove unused imports
50848ff
@manticore-projects
Merge branch 'master' of github.com:JSQLParser/JSqlParser
608a6ef
@manticore-projects
Merge github.com:JSQLParser/JSqlParser
bb3a9f1
@manticore-projects
Merge remote-tracking branch 'manticore/master'
15e713b
@manticore-projects
Support Postgres INSERT ... ON CONFLICT
92c8937 
Fixes JSQLParser#1551
Refactor UpdateSet.toString(), which is used by Insert and Update
@manticore-projects
Allow KEEP keyword
44809d4 
Enables special Oracle Test keywordasidentifier04.sql, now 191 tests succeed
@manticore-projects
Sanitize before push
80e3f07
@manticore-projects
Merge remote-tracking branch 'origin/master' into InsertOnConflict
50a8d26
@manticore-projects

manticore-projects commented Jun 3, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor Author

I have no idea, why the Maven Build fails when the Gradle Build compiles and tests fine.
The generated sources have differences (maven on left and gradle on the right):
image

After a bit of trial and error, I think MAVEN needs to set the following JavaCC options:
arguments = [grammar_encoding: 'UTF-8', static: 'false', java_template_type: 'modern']

(Without those, the Gradle Build also fails.)
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@manticore-projects
Tweak Grammar in order to survive the Maven Build
5f7da94 
Ammend the README
@manticore-projects

manticore-projects commented Jun 3, 2022

Copy link
Copy Markdown
Contributor Author

Very strange. The following Production would compile with the Gradle Plugin, but not with Maven:

Test myTest(): 
{
   Test result;
}
{
    (
      result=TestProduction1() { return result; }
    )
    |
    (
      result=TestProduction2() 
    ) { return result; }
}

Rewriting it solves the problem:

Test myTest(): 
{
   Test result;
}
{
    (
        (
          result=TestProduction1() { return result; }
        )
        |
        (
          result=TestProduction2() 
        )
    ) { return result; }
}
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@manticore-projects manticore-projects mentioned this pull request Jun 3, 2022
Production works with the Gradle Plugin, but fails with the Maven Plugin javacc/javacc#229
Closed
manticore-projects added 4 commits June 4, 2022 15:38
@manticore-projects
Move Plugin configuration files to the CONFIG folder (hoping, that Co…
35ccf0b 
…dacy will find it there)

Update PMD in the Maven configuration
@manticore-projects
Update PMD in the Maven and Gradle configuration
203076f
@manticore-projects
Merge remote-tracking branch 'origin/master' into InsertOnConflict
fadd055 
# Conflicts:
#	README.md
@manticore-projects
Appease Codacy
07f1950
@wumpz
Merge branch 'master' into InsertOnConflict
760045e
@wumpz
wumpz merged commit 5ae09ad into JSQLParser:master Jul 19, 2022
@manticore-projects
manticore-projects deleted the InsertOnConflict branch July 19, 2022 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

postgres sql parsing "conflict " error

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@manticore-projects @wumpz
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.