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.

Sign up for GitHub

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

Jump to bottom

clg reports SQLitePreparedStatement operation failure after printing one log message #266

Closed
wraymo opened this issue Feb 6, 2024 · 0 comments
Closed

clg reports SQLitePreparedStatement operation failure after printing one log message #266

wraymo opened this issue Feb 6, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@wraymo
Copy link
Contributor

wraymo commented Feb 6, 2024 •
edited
Loading

Bug

The current version of clg can cause SQLitePreparedStatement: Failed to bind int64 to statement - column index out of range bug. The reason is due to implicit conversion from segment_id to bool when calling clp::streaming_archive::reader::Archive::get_file_iterator for false == is_superseding_query case.

The fix is pretty simple. We just need to add false as the fifth parameter of get_file_iterator so that it can match the correct method.

CLP version

59b2c22

Environment

MacOS 13.2.1

Reproduction steps

  • Compress components/core/README.md using clp
  • Search option on the compressed archive using clg
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@wraymo