[idempiere/idempiere] 0d5dd8: IDEMPIERE-7023 Add ISQLStatementRewriter extension...

0 views
Skip to first unread message

Stefano Coletta

unread,
Jul 3, 2026, 6:28:02 PM (6 days ago) Jul 3
to idempi...@googlegroups.com
Branch: refs/heads/master
Home: https://github.com/idempiere/idempiere
Commit: 0d5dd8219d92da7edb6eeabd4f12f499644228fe
https://github.com/idempiere/idempiere/commit/0d5dd8219d92da7edb6eeabd4f12f499644228fe
Author: Stefano Coletta <105856436+...@users.noreply.github.com>
Date: 2026-07-04 (Sat, 04 Jul 2026)

Changed paths:
M org.adempiere.base/src/org/compiere/dbPort/Convert.java
A org.adempiere.base/src/org/compiere/dbPort/ISQLStatementRewriter.java
A org.adempiere.base/src/org/compiere/dbPort/SQLStatementRewriterProvider.java
M org.compiere.db.oracle.provider/src/org/compiere/db/DB_Oracle.java
M org.compiere.db.postgresql.provider/src/org/compiere/db/DB_PostgreSQL.java

Log Message:
-----------
IDEMPIERE-7023 Add ISQLStatementRewriter extension point in Convert / DB providers (#3279)

* IDEMPIERE-7023 Add ISQLStatementRewriter extension point in Convert / DB providers

Adds a new OSGi service interface org.compiere.dbPort.ISQLStatementRewriter
that lets a plugin pre-rewrite SQL statements before Convert.convert() and
dialect convertStatement() execute them.

* IDEMPIERE-7023 SqlRewriter: ensure rewriter runs once per statement

* IDEMPIERE-7023 Remove ThreadLocal guard from SQLStatementRewriter; re-entrancy is now prevented by contract on ISQLStatementRewriter

* IDEMPIERE-7023 Remove ThreadLocal guard and Service.locator from Convert; delegate to SQLStatementRewriter

* IDEMPIERE-7023 Rename SQLStatementRewriter to SQLStatementRewriterProvider to remove confusion with ISQLStatementRewriter implementations

* IDEMPIERE-7023 Rename SQLStatementRewriter to SQLStatementRewriterProvider

* IDEMPIERE-7023 Remove delegate methods from Convert, call SQLStatementRewriterProvider directly



To unsubscribe from these emails, change your notification settings at https://github.com/idempiere/idempiere/settings/notifications
Reply all
Reply to author
Forward
0 new messages