You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@seata.apache.org by "w1397800 (via GitHub)" <gi...@apache.org> on 2025/08/11 10:09:20 UTC
[I] Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration [incubator-seata]
w1397800 opened a new issue, #7581:
URL: https://github.com/apache/incubator-seata/issues/7581
### Check Ahead
- [x] I have searched the [issues](https://github.com/seata/seata/issues) of this repository and believe that this is not a duplicate.
- [x] I am willing to try to fix this bug myself.
### Ⅰ. Issue Description
I found that in the PostgresqlUndoLogManager class, the sequence name undo_log_id_seq used in the insert SQL statement is hardcoded:
`INSERT INTO undo_log (...) VALUES (nextval('undo_log_id_seq'), ...)`
This causes an issue when the undo log table name is customized via configuration (e.g., in application.yml). Because the sequence name does not update to match the new table name, insert operations fail or behave incorrectly.
Steps to reproduce:
1. Configure Seata to use a custom undo log table name, for example my_undo_log.
2. The sequence name remains hardcoded as undo_log_id_seq in the SQL.
3. On insert, PostgreSQL throws an error because the sequence undo_log_id_seq does not exist or does not match the new table.
Expected behavior:
- The sequence name used in the insert SQL should be dynamically derived from the configured undo log table name, for example my_undo_log_id_seq for table my_undo_log.
- This would make the undo log manager compatible with table name customization.
Suggested fix:
- Make the sequence name configurable or derived from the undo log table name dynamically.
- Update the insert SQL statement in PostgresqlUndoLogManager accordingly.
### Ⅱ. Describe what happened
_No response_
### Ⅲ. Describe what you expected to happen
_No response_
### Ⅳ. How to reproduce it (as minimally and precisely as possible)
_No response_
### Ⅴ. Anything else we need to know?
_No response_
### Ⅵ. Environment
_No response_
--
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: notifications-unsubscribe@seata.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org
Re: [I] Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration [incubator-seata]
Posted by "maple525866 (via GitHub)" <gi...@apache.org>.
maple525866 commented on issue #7581:
URL: https://github.com/apache/incubator-seata/issues/7581#issuecomment-3418451429
Excuse me, are you still paying attention to the work on this issue recently? If not, could you transfer it to me? @songxiaosheng Thank you!
--
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: notifications-unsubscribe@seata.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org
Re: [I] Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration [incubator-seata]
Posted by "funky-eyes (via GitHub)" <gi...@apache.org>.
funky-eyes commented on issue #7581:
URL: https://github.com/apache/incubator-seata/issues/7581#issuecomment-3226342826
I confirm this issue exists. Can you submit a PR or provide a detailed solution for discussion?
--
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: notifications-unsubscribe@seata.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org
Re: [I] Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration [incubator-seata]
Posted by "songxiaosheng (via GitHub)" <gi...@apache.org>.
songxiaosheng commented on issue #7581:
URL: https://github.com/apache/incubator-seata/issues/7581#issuecomment-3277063340
assign me
--
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: notifications-unsubscribe@seata.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org
Re: [I] Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration [incubator-seata]
Posted by "YvCeung (via GitHub)" <gi...@apache.org>.
YvCeung closed issue #7581: Sequence name `undo_log_id_seq` in `PostgresqlUndoLogManager` breaks custom `undo_log` table configuration
URL: https://github.com/apache/incubator-seata/issues/7581
--
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: notifications-unsubscribe@seata.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@seata.apache.org
For additional commands, e-mail: notifications-help@seata.apache.org