Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IO stats unit test fixes #1896

Merged
merged 6 commits into from
Dec 13, 2023
Merged

IO stats unit test fixes #1896

merged 6 commits into from
Dec 13, 2023

Conversation

Thejas-bhat
Copy link
Member

@Thejas-bhat Thejas-bhat commented Nov 3, 2023

  • the recent index sections format uncovered some bugs in the io stats computation - specifically around the docValues area.
  • furthermore, there is additional metadata being read as part of format changes - which is the fieldsSection index.

go.mod Outdated
)

replace github.com/blevesearch/zapx/v16 => ../zapx
Copy link
Member

Choose a reason for hiding this comment

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

You can remove this and use the branch refactor-section for zapx/v16 before running go mod tidy again. This is until we merge refactor-section into master.

@Thejas-bhat Thejas-bhat changed the base branch from master to unstable November 3, 2023 14:53
@Thejas-bhat Thejas-bhat force-pushed the refactor-sections branch 3 times, most recently from 6c77b83 to 32bd6bc Compare November 28, 2023 10:07
@Thejas-bhat Thejas-bhat changed the title segment plugin changes for the v16 zap module IO stats unit test fixes Dec 6, 2023
@abhinavdangeti
Copy link
Member

--- FAIL: TestBytesWritten (1.20s)
Error:     index_test.go:338: expected bytes written is 78985, got 147850

@abhinavdangeti abhinavdangeti merged commit 4f0fa6b into unstable Dec 13, 2023
9 checks passed
@abhinavdangeti abhinavdangeti deleted the refactor-sections branch December 13, 2023 16:35
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