Skip to content

FINERACT-2691: Bind group summary report parameters as numeric#6139

Merged
IOhacker merged 2 commits into
apache:developfrom
AnvayKharb:fix/FINERACT-2691-group-summary-postgresql-parameter
Jul 24, 2026
Merged

FINERACT-2691: Bind group summary report parameters as numeric#6139
IOhacker merged 2 commits into
apache:developfrom
AnvayKharb:fix/FINERACT-2691-group-summary-postgresql-parameter

Conversation

@AnvayKharb

Copy link
Copy Markdown
Contributor

Summary

Fixes PostgreSQL failures in group summary reports by adding the missing numeric report parameter mappings for groupId.

This ensures R_groupId is bound as a numeric value instead of a string, allowing the reports to execute correctly on PostgreSQL while preserving existing behavior for MariaDB and MySQL.

The fix also includes a regression test covering the affected group summary report endpoints.

Related issues and discussion

FINERACT-2691

@AnvayKharb AnvayKharb changed the title fix(FINERACT-2691): bind group summary report parameters as numeric FINERACT-2691: Bind group summary report parameters as numeric Jul 16, 2026
@AnvayKharb
AnvayKharb force-pushed the fix/FINERACT-2691-group-summary-postgresql-parameter branch 4 times, most recently from fb72b91 to 7f4eb31 Compare July 18, 2026 14:27
@IOhacker

Copy link
Copy Markdown
Contributor

@AnvayKharb make sure to sign the commit please

@AnvayKharb
AnvayKharb force-pushed the fix/FINERACT-2691-group-summary-postgresql-parameter branch from 7f4eb31 to 863272f Compare July 21, 2026 19:37

@IOhacker IOhacker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AnvayKharb

Copy link
Copy Markdown
Contributor Author

Hi @adamsaghy @vidakovic please review

@IOhacker
IOhacker merged commit 310d938 into apache:develop Jul 24, 2026
91 checks passed
@AnvayKharb
AnvayKharb deleted the fix/FINERACT-2691-group-summary-postgresql-parameter branch July 24, 2026 14:23
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