-
Notifications
You must be signed in to change notification settings - Fork 116
PostgreSQL-specific ltrim
and rtrim
#341
Comments
Probably we should move these functions from the core module to modules specific for every database servers. |
This is still an open issue. Research is needed to find out if ltrim and rtrim behave differently on different databases. If yes, then move them to specific modules and implement accordingly. |
It would be best if there is a way to override in the DB-specific module the default implementation from core. That way, even if |
The unary variants of
So I'd leave Note that while SQL Server since 2022 (compatibility level 160) supports Side note to the naming: There is some inconsistency in the casing of |
#341 - Implement Ltrim2 and Rtrim2 for Postgres and Oracle
As @marekklis pointed out in #339, it appears that PostgreSQL can take an additional, optional parameter in
ltrim
andrtrim
functions.Just adding these functions in PostgresModule might cause import clashes. We need to figure out the way to deal with such situation.
The text was updated successfully, but these errors were encountered: