fix: make sure the table have defined primary keys when using updateable resultset #2101

Fzhlib · 2021-03-15T14:14:24Z

Fix for #1975

fix: make sure the table have defined primary keys when using updateable resultset
test: add test for updateable resultset without primary key

All Submissions:

Have you followed the guidelines in our Contributing document?
Have you checked to ensure there aren't other open Pull Requests for the same update/change?

…ble resultset

pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSet.java

…ble resultset

Fzhlib · 2021-03-18T11:28:46Z

Hi @davecramer, Is there anything else i should do?

…ble resultset (pgjdbc#2101) * fix: make sure the table have defined primary keys when using updateable resultset * test: add test for updateable resultset without primary key Co-authored-by: System Administrator <root@192.168.0.15> Co-authored-by: Fe <fe@192.168.0.15>

…ble resultset (#2101) (#2106) * fix: make sure the table have defined primary keys when using updateable resultset * test: add test for updateable resultset without primary key Co-authored-by: Feng zhihao <Fzhlib@users.noreply.github.com>

System Administrator added 2 commits March 15, 2021 21:58

fix: make sure the table have defined primary keys when using updatea…

9272880

…ble resultset

test: add test for updateable resultset without primary key

2c76a9d

davecramer reviewed Mar 15, 2021

View reviewed changes

pgjdbc/src/main/java/org/postgresql/jdbc/PgResultSet.java Outdated Show resolved Hide resolved

fix: make sure the table have defined primary keys when using updatea…

553cf2e

…ble resultset

davecramer merged commit 66b1e02 into pgjdbc:master Mar 22, 2021

Fzhlib deleted the fix_isUpdateable branch March 23, 2021 09:08

sync-by-unito bot mentioned this pull request Jun 11, 2021

Bump postgresql from 42.2.20 to 42.2.21 liquibase/liquibase-postgresql#60

Merged

sync-by-unito bot mentioned this pull request Jul 7, 2021

Bump postgresql from 42.2.21 to 42.2.23 liquibase/liquibase-postgresql#64

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: make sure the table have defined primary keys when using updateable resultset #2101

fix: make sure the table have defined primary keys when using updateable resultset #2101

Fzhlib commented Mar 15, 2021 •

edited

Fzhlib commented Mar 18, 2021

fix: make sure the table have defined primary keys when using updateable resultset #2101

fix: make sure the table have defined primary keys when using updateable resultset #2101

Conversation

Fzhlib commented Mar 15, 2021 • edited

All Submissions:

Fzhlib commented Mar 18, 2021

Fzhlib commented Mar 15, 2021 •

edited