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.

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

sql: JDBC wants "set session parameters as ..." #4524

Closed
knz opened this issue Feb 19, 2016 · 2 comments
Closed

sql: JDBC wants "set session parameters as ..." #4524

knz opened this issue Feb 19, 2016 · 2 comments

Comments

@knz
Copy link
Contributor

knz commented Feb 19, 2016

Using the same test: #3819 (comment)

Now fails with:

ERROR in (a-test) (QueryExecutorImpl.java:2182)
Uncaught exception, not in assertion.
expected: nil
  actual: org.postgresql.util.PSQLException: ERROR: syntax error at or near "AS"
SET SESSION CHARACTERISTICS AS TRANSACTION ISOLATION LEVEL SERIALIZABLE
                            ^

 at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse (QueryExecutorImpl.java:2182)
    org.postgresql.core.v3.QueryExecutorImpl.processResults (QueryExecutorImpl.java:1911)
    org.postgresql.core.v3.QueryExecutorImpl.execute (QueryExecutorImpl.java:173)
    org.postgresql.jdbc2.AbstractJdbc2Statement.execute (AbstractJdbc2Statement.java:616)
    org.postgresql.jdbc2.AbstractJdbc2Statement.executeWithFlags (AbstractJdbc2Statement.java:452)
    org.postgresql.jdbc2.AbstractJdbc2Connection.execSQLUpdate (AbstractJdbc2Connection.java:399)
    org.postgresql.jdbc2.AbstractJdbc2Connection.setTransactionIsolation (AbstractJdbc2Connection.java:976)
    clojure.java.jdbc$db_transaction_STAR_.doInvoke (jdbc.clj:582)
    clojure.lang.RestFn.invoke (RestFn.java:464)
    jdbc_test.core_test/fn (core_test.clj:16)
@petermattis
Copy link
Collaborator

SET SESSION CHARACTERISTICS AS TRANSACTION ISOLATION LEVEL is a verbose equivalent to SET DEFAULT_TRANSACTION_ISOLATION. Now that we support SHOW DEFAULT_TRANSACTION_ISOLATION we should probably also support SET.

@knz
Copy link
Contributor Author

knz commented Feb 25, 2016

While looking at #4036

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

No branches or pull requests

2 participants