Copilot commented on code in PR #6709: URL: https://github.com/apache/texera/pull/6709#discussion_r3626721062
########## project/plugins.sbt: ########## @@ -40,5 +40,5 @@ libraryDependencies ++= Seq( // PostgresDatabase.loadForeignKeys can't read (it looks up lowercase key_seq), // breaking JOOQ code generation. Fixed only in jOOQ 3.20+ (jOOQ/jOOQ#17873); jOOQ // is capped at 3.19.36 here (the last Java-17 release), so keep this driver pinned. - "org.postgresql" % "postgresql" % "42.7.4" + "org.postgresql" % "postgresql" % "42.7.13" Review Comment: This bumps the build-time pgjdbc used for jOOQ code generation to 42.7.13, but the adjacent rationale says pgjdbc >= 42.7.5 breaks jOOQ 3.19.x codegen due to JDBC metadata column-label casing (KEY_SEQ vs key_seq). With jOOQ still pinned at 3.19.36 here, upgrading the driver likely reintroduces the codegen failure the comment is warning about; either keep pgjdbc <= 42.7.4 or upgrade jOOQ to a version that fixes jOOQ/jOOQ#17873. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
