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

Fix two embedded usage pitfalls #51

Closed
HouzuoGuo opened this issue Feb 13, 2014 · 3 comments
Closed

Fix two embedded usage pitfalls #51

HouzuoGuo opened this issue Feb 13, 2014 · 3 comments
Assignees
@HouzuoGuo

Comments

@HouzuoGuo
Copy link
Owner

Pitfall : User may forget to initialize random number generator
Proposed fix: Initialize the Go pseudo random number generator by feeding seed from crypto/rand upon opening a database

Pitfall 2: Type conversion failure caused missing index content.
Proposed fix: Log an error message when such type conversion failure occurs.
@HouzuoGuo HouzuoGuo self-assigned this Feb 13, 2014
@HouzuoGuo
Copy link
Owner Author

Pitfall 1 has been fixed in "nextgen" branch.

Pitfall 2 is still WIP.

@HouzuoGuo
Copy link
Owner Author

2nd: will be fixed in nextgen branch

@HouzuoGuo
Copy link
Owner Author

All fixed

2nd: fixed in 0aa5cdb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HouzuoGuo HouzuoGuo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@HouzuoGuo