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

"failed to fill whole buffer" errors #14

Closed
mitya52 opened this issue Apr 30, 2022 · 2 comments
Closed

"failed to fill whole buffer" errors #14

mitya52 opened this issue Apr 30, 2022 · 2 comments

Comments

@mitya52
Copy link

mitya52 commented Apr 30, 2022

Hi,

I have tried to run the code on simple string and count-occurrences fails with "failed to fill whole buffer" error.

Here are steps to reproduce:

  1. run ./target/debug/dedup_dataset make --data-file dup.txt, data file dup.txt contains simple string "aaabbb"
  2. then run ./target/debug/dedup_dataset count-occurrences --data-file dup.txt --query-file query.txt, where query.txt contains
  • "bb"
    expectation: Number of times present: 2
    reality: thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { kind: UnexpectedEof, message: "failed to fill whole buffer" }', src/main.rs:275:31;
  • "ab"
    expectation: Number of times present: 1
    reality: thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Error { kind: UnexpectedEof, message: "failed to fill whole buffer" }', src/main.rs:297:31;
  • "b"
    expectation: Number of times present: 2
    reality: Number of times present: 1;

May be I'm doing something wrong? Thanks.

@carlini carlini closed this as completed in 7c1544c May 1, 2022
@carlini
Copy link
Collaborator

carlini commented May 1, 2022

Ah sorry this is a bug with something from the rewrite. I've updated to the new version of the code if you pull -- please let me know if it works correctly.

@mitya52
Copy link
Author

mitya52 commented May 1, 2022

Thanks, all "failed to fill whole buffer" errors gone with fix!

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

No branches or pull requests

2 participants