couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Paul Joseph Davis (JIRA)" <j...@apache.org>
Subject [jira] Commented: (COUCHDB-542) Fix for COUCHDB-288 makes JS_MaybeGC not called
Date Sun, 25 Oct 2009 17:23:59 GMT

    [ https://issues.apache.org/jira/browse/COUCHDB-542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12769836#action_12769836
] 

Paul Joseph Davis commented on COUCHDB-542:
-------------------------------------------

The view servers are run in their own OS level process and communicated with over stdio. Thus
when we kill the process its up to the kernel to make sure all is well. I'm toying around
with refactoring couchjs so that I can work on the command line tests. I'll look at making
those changes as well and see how things go.

> Fix for COUCHDB-288 makes JS_MaybeGC not called
> -----------------------------------------------
>
>                 Key: COUCHDB-542
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-542
>             Project: CouchDB
>          Issue Type: Bug
>          Components: JavaScript View Server
>            Reporter: Mike Hommey
>
> The fix for COUCHDB-288 only basically replaces JS_SetBranchCallback with JS_SetOperationCallback,
which is not enough for the callback to be triggered. The problem is that basically, the operation
callback API has now nothing to do with the previous branch callback API, and is not called
at each branch call at the JS level. Actually, it is not called at all, except if JS_TriggerOperationCallback
is used. Typically, this needs to be done either from a signal handler or a watchdog thread,
in which case the test inside the callback is pretty pointless.
> See http://groups.google.com/group/mozilla.dev.tech.js-engine/browse_thread/thread/a4d1fe147761aacb/e61d2592faf4ef72?lnk=gst&q=js_setoperationcallback#e61d2592faf4ef72
for reference.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message