Skip to content

Remove useGeneratedKeys from MyBatis mappers incompatible with SQLite JDBC#303

Draft
Copilot wants to merge 2 commits intodevelopfrom
copilot/resolve-target-audience-issue
Draft

Remove useGeneratedKeys from MyBatis mappers incompatible with SQLite JDBC#303
Copilot wants to merge 2 commits intodevelopfrom
copilot/resolve-target-audience-issue

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 15, 2026

SQLite JDBC driver does not implement Statement.getGeneratedKeys(), causing SQLFeatureNotSupportedException on every insert to t_people, t_task, and t_task_his.

  • Removed useGeneratedKeys="true" and keyProperty="id" from <insert> statements in TPeopleMapper.xml, TTaskMapper.xml, and TTaskHisMapper.xml
  • No callers read back the generated key from the domain object after insert, so the attributes were unnecessary
  • The id column is already explicitly included in the insert SQL and bound via #{id,jdbcType=INTEGER}

Before:

<insert id="insert" parameterType="...TPeople" keyProperty="id" useGeneratedKeys="true">

After:

<insert id="insert" parameterType="...TPeople">

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • jitpack.io
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.14/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.14/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.14 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.14/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/WePush/WePush org.codehaus.plexus.classworlds.launcher.Launcher compile -q (dns block)
  • mvnrepository.com
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.14/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.14/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.14 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.14/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/WePush/WePush org.codehaus.plexus.classworlds.launcher.Launcher compile -q (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

…ite compatibility

SQLite JDBC driver does not support the getGeneratedKeys() feature.
Removed useGeneratedKeys="true" and keyProperty="id" from insert
statements in TPeopleMapper.xml, TTaskMapper.xml, and TTaskHisMapper.xml
to resolve SQLFeatureNotSupportedException errors.

Agent-Logs-Url: https://github.com/rememberber/WePush/sessions/4faf8787-3ae1-428f-8f7c-af1bf9d65878

Co-authored-by: rememberber <4653893+rememberber@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue in target audience preparation stage Remove useGeneratedKeys from MyBatis mappers incompatible with SQLite JDBC Apr 15, 2026
Copilot AI requested a review from rememberber April 15, 2026 16:19
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