Skip to content

GitHub Issue 1219: Summary statistics fail with "Character read limit of 100000 exceeded" on wide datasets (Backport) - #7940

Merged
cnathe merged 1 commit into
release26.7-SNAPSHOTfrom
26.7_fb_backport1219
Aug 17, 2026
Merged

GitHub Issue 1219: Summary statistics fail with "Character read limit of 100000 exceeded" on wide datasets (Backport)#7940
cnathe merged 1 commit into
release26.7-SNAPSHOTfrom
26.7_fb_backport1219

Conversation

@cnathe

@cnathe cnathe commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Rationale

Backport changes to 26.7 for https://github.com/LabKey/internal-issues/issues/1219

On wide grids (many columns), setting a summary statistic posts the full custom-view JSON to the saveQueryViews action, which is capped at @JsonInputLimit(100_000) characters. The per-column field metadata array (fields) plus full column objects blow past that limit and the save fails ("Character read limit of 100000 exceeded"). The fix trims the POST body in _updateSessionCustomView: drop the unused top-level fields array and reduce each column to just fieldKey (+ title when present).

Related Pull Requests

Changes

  • DataRegion call to saveQueryViews does not need to include view 'fields' info in post

… include view 'fields' info in post

- backport from develop
@cnathe
cnathe merged commit 27a006a into release26.7-SNAPSHOT Aug 17, 2026
10 of 11 checks passed
@cnathe
cnathe deleted the 26.7_fb_backport1219 branch August 17, 2026 19:35
cnathe added a commit to LabKey/testAutomation that referenced this pull request Aug 17, 2026
… of 100000 exceeded" on wide datasets (Backport) (#3154)

## Rationale
LabKey/internal-issues#1219

See related PR for rationale. This PR adds the test case.

## Related Pull Requests
- #3081
- LabKey/platform#7940
- #3154

## Changes
- CustomizeViewTest test case for saveQueryView with wide column list in
view
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