Skip to content

[DOC-14757]: Feedback on bleveMaxClauseCount – Couchbase Docs - #4211

Merged
RayOffiah merged 3 commits into
release/8.0from
DOC-14757--8.0--Feedback-on-bleveMaxClauseCount--Couchbase-Docs
Oct 9, 2026
Merged

RayOffiah merged 3 commits into
release/8.0from
DOC-14757--8.0--Feedback-on-bleveMaxClauseCount--Couchbase-Docs

Conversation

@RayOffiah

Copy link
Copy Markdown
Contributor

Description

This pull request updates the documentation concerning bleveMaxClauseCount. It clarifies its default behavior, emphasizes query optimization, and introduces enhancements for retrieving cluster-level options:

  • Added a "Getting Cluster Level Options" section, along with CURL examples, to fts-cluster-options.adoc.
  • Improved documentation for bleveMaxResultWindow and maxReplicasAllowed with reusable steps.
  • Added a new partial for ease of cluster-level option retrieval.

Preview URL:
https://preview.docs-test.couchbase.com/docs-server-DOC-14757--8.0--Feedback-on-bleveMaxClauseCount--Couchbase-Docs

You will need the Docs Team credentials on Confluence.

- Updated documentation for `bleveMaxClauseCount` to clarify default behavior and emphasize query optimization.
- Included new partial for retrieving cluster-level options using CURL.
- Added "Getting Cluster Level Options" section and examples to `fts-cluster-options.adoc`.
- Enhanced `bleveMaxResultWindow` and `maxReplicasAllowed` documentation with reusable cluster-level retrieval steps
@RayOffiah
RayOffiah requested a balanced review from Copilot October 9, 2026 11:00
@RayOffiah RayOffiah self-assigned this Oct 9, 2026

Copilot AI 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.

🟡 Changes recommended

Several documentation passages contain incomplete, fragmented, or inaccurate descriptions.

4 open findings
What changed in this PR

Updates Full Text Search cluster-option documentation with retrieval guidance and clearer configuration details.

Changes:

  • Adds reusable cluster-option retrieval instructions.
  • Documents the /api/conciseOptions response.
  • Clarifies query optimization and selected option guidance.
File Description
modules/​fts/​partials/​get-cluster-level-options.adoc Adds reusable retrieval command.
modules/​fts/​pages/​fts-cluster-options.adoc Adds retrieval section and response example.
modules/​fts/​pages/​fts-advanced-settings-maxReplicasAllowed.adoc Includes retrieval guidance.
modules/​fts/​pages/​fts-advanced-settings-bleveMaxResultWindow.adoc Revises explanation and includes retrieval guidance.
modules/​fts/​pages/​fts-advanced-settings-bleveMaxClauseCount.adoc Clarifies query guidance and includes retrieval instructions.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread modules/fts/pages/fts-advanced-settings-bleveMaxClauseCount.adoc Outdated
Comment thread modules/fts/pages/fts-advanced-settings-bleveMaxResultWindow.adoc Outdated
Comment thread modules/fts/pages/fts-cluster-options.adoc Outdated
Comment thread modules/fts/pages/fts-cluster-options.adoc Outdated
- Updated `bleveMaxClauseCount` documentation to correct spacing and reinforce query optimization best practices.
- Refined `bleveMaxResultWindow` description by removing an incomplete sentence for clarity.
- Edited `fts-cluster-options.adoc` to improve wording and ensure consistency, including specifying JSON response format.
@RayOffiah
RayOffiah requested a balanced review from Copilot October 9, 2026 11:15
@RayOffiah
RayOffiah requested a review from a team October 9, 2026 11:15

Copilot AI 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.

🟡 Changes recommended

The new reusable command transmits cluster credentials over unencrypted HTTP.

1 open finding
4 resolved since last review

🧠 Review effort: Balanced

Comment thread modules/fts/partials/get-cluster-level-options.adoc Outdated
…ials over unencrypted HTTP'

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI 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.

🔵 Needs a closer look

The retrieval endpoint lacks a documented public contract, and the sample contradicts the stated default clause count.

0 open findings

1 resolved since last review
Previously missed (2)

In code that hasn't changed since last review

Low severity Correct bleveMaxClauseCount default in example

modules/​fts/​pages/​fts-cluster-options.adoc:41

This sample shows bleveMaxClauseCount as 10000, while the linked settings page states that its default is 1024. Because this standalone section does not first configure an override, the example should show 1024 (or explicitly explain that it comes from a modified cluster) so users checking the default do not receive contradictory guidance.

Low severity Document or replace unsupported conciseOptions endpoint

modules/​fts/​partials/​get-cluster-level-options.adoc:8

The Search Service REST inventory documents GET /api/managerOptions (modules/rest-api/partials/rest-search-service-table.adoc:253-261), but contains no /api/conciseOptions contract. Introducing this undocumented endpoint makes the retrieval instructions dependent on an API whose supported versions and response contract are unclear. Please either use the supported retrieval endpoint and update the sample response, or add the corresponding REST API documentation/version requirements for /api/conciseOptions.

🧠 Review effort: Balanced

@RayOffiah
RayOffiah requested a review from Samsonnyyeet October 9, 2026 12:07
@Pallavi-Janardhan
Pallavi-Janardhan self-requested a review October 9, 2026 12:29

@Pallavi-Janardhan Pallavi-Janardhan 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

@sarahlwelton

Copy link
Copy Markdown
Contributor

@RayOffiah , I thought we were going to make the team check in about the missing API option before doing this. I'm not sure that conciseOptions is an endpoint on the API and I'm concerned this is hallucinated.

@RayOffiah
RayOffiah merged commit 90b2abc into release/8.0 Oct 9, 2026
1 check passed
@RayOffiah
RayOffiah deleted the DOC-14757--8.0--Feedback-on-bleveMaxClauseCount--Couchbase-Docs branch October 9, 2026 13:32
@RayOffiah
RayOffiah restored the DOC-14757--8.0--Feedback-on-bleveMaxClauseCount--Couchbase-Docs branch October 9, 2026 15:47
RayOffiah added a commit that referenced this pull request Oct 9, 2026
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.

4 participants