Skip to content

Conversation

@sbernabe1
Copy link

Updating doc's to mention the following below:

Setting the page-size to 0 is recommended to disable query paging, avoiding unexpected gaps in graph data.

Setting the page-size to 0 is recommended to disable query paging, avoiding unexpected gaps in graph data.
@github-actions github-actions bot added the docs label Nov 6, 2025
@github-actions github-actions bot requested a review from indigo423 November 6, 2025 20:20
@dino2gnt dino2gnt self-requested a review November 6, 2025 21:12
@dino2gnt
Copy link
Contributor

dino2gnt commented Nov 6, 2025

Ship it

dino2gnt
dino2gnt previously approved these changes Nov 6, 2025
@cgorantla
Copy link
Contributor

@sbernabe1 Are we ready to merge this ? Just curious, Is there some more explanation on why this recommendation has changed.

@dino2gnt
Copy link
Contributor

dino2gnt commented Dec 5, 2025

@sbernabe1 Are we ready to merge this ? Just curious, Is there some more explanation on why this recommendation has changed.

@cgorantla
AFAIK, OpenNMS doesn't have any support for the pagination added in the Cassandra 4.x client. Previously, we specified a large value for page-size and crossed our fingers that it was large enough to hold the entire reply. However, Support encountered a situation where our previous recommendation was not enough, and in fact even going upwards of 1million was still not holding the entire reply. Saul discovered that page-size=0 was available to turn off the pagination entirely, which fixed the problem for the customer, and so we're updating the documentation to reflect the new information.

@sbernabe1 sbernabe1 changed the base branch from release-34.x to release-35.x December 23, 2025 16:46
@sbernabe1 sbernabe1 dismissed dino2gnt’s stale review December 23, 2025 16:46

The base branch was changed.

@sbernabe1 sbernabe1 changed the base branch from release-35.x to foundation-2024 December 23, 2025 16:50
@sbernabe1 sbernabe1 changed the base branch from foundation-2024 to release-35.x December 23, 2025 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants