AshharAhmadKhan commented on PR #5452:
URL: https://github.com/apache/fineract/pull/5452#issuecomment-3859551353

   Updated to use `current_schema()` with explicit function call syntax for 
improved reliability across PostgreSQL versions.
   
   **Changes made:**
   - Added parentheses to 3 occurrences of `current_schema` in 
`PostgreSQLQueryService.java`:
     - `isTablePresent()` method
     - `getTableColumns()` method  
     - `getTableIndexes()` method
   - This ensures consistent schema resolution for tenant-specific PostgreSQL 
deployments
   
   **Regarding the CI failure:**
   The previous test failure (`m_command table already exists` error in 
Liquibase) appears to occur during database initialization, before datatable 
operations run. Since this change only affects datatable metadata queries in 
`PostgreSQLQueryService`, the failure may be unrelated to this fix. Happy to 
investigate further if the issue persists after the CI retry.
   
   Thank you for your patience and review!


-- 
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]

Reply via email to