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

mongodb: pin non-native instances to avoid GC crashes #1855

Merged
merged 1 commit into from
Nov 29, 2015

Conversation

Morriar
Copy link
Member

@Morriar Morriar commented Nov 28, 2015

Signed-off-by: Alexandre Terrasa alexandre@moz-code.org

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>
if json isa JsonParseError then
print to_s
print json.message
sys.exit 1
Copy link
Member

Choose a reason for hiding this comment

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

for information: sys. is superfluous here

@privat
Copy link
Member

privat commented Nov 28, 2015

+1

privat added a commit that referenced this pull request Nov 28, 2015
Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Pull-Request: #1855
Reviewed-by: Jean Privat <jean@pryen.org>
@privat privat merged commit 30aaf3c into nitlang:master Nov 29, 2015
@Morriar Morriar deleted the fix-mongo branch November 29, 2015 03:40
privat added a commit that referenced this pull request Nov 30, 2015
This PR add two more fixes since #1855 :
* use `FinalizableOnce` instead of `Finalizable`
* copy bson structs returned by `mongoc_cursor_item`

Pull-Request: #1866
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants