Skip to content

Conversation

@abhinavdangeti
Copy link
Member

No description provided.

abhinavdangeti added a commit to blevesearch/scorch_segment_api that referenced this pull request Aug 18, 2025
@abhinavdangeti abhinavdangeti changed the title MB-66396: Interfaces for "InsightsIndex" MB-66396: Interface for "InsightsIndex" Aug 18, 2025
@abhinavdangeti abhinavdangeti changed the title MB-66396: Interface for "InsightsIndex" MB-66396: Interface for InsightsIndex Aug 18, 2025
abhinavdangeti added a commit to blevesearch/scorch_segment_api that referenced this pull request Aug 18, 2025
@abhinavdangeti abhinavdangeti changed the title MB-66396: Interface for InsightsIndex MB-66396: Interface for IndexInsightsReader Aug 18, 2025
abhinavdangeti added a commit to blevesearch/scorch_segment_api that referenced this pull request Aug 18, 2025
@abhinavdangeti abhinavdangeti marked this pull request as ready for review August 22, 2025 21:16
Comment on lines +370 to +374

type TermFreq struct {
Term string `json:"term"`
Frequency uint64 `json:"frequency"`
}
Copy link
Member

Choose a reason for hiding this comment

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

not required, the struct required is already defined, this becomes duplicate Count === Frequency

type DictEntry struct {
	Term         string
	Count        uint64
	EditDistance uint8
}

Copy link
Member Author

Choose a reason for hiding this comment

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

Fair, just created a separate entry for context.

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