Skip to content
This repository was archived by the owner on Aug 13, 2026. It is now read-only.

Add app_env var for script timeout - #33

Closed
russelldb wants to merge 1 commit into
masterfrom
rdb-js-timeout
Closed

Add app_env var for script timeout#33
russelldb wants to merge 1 commit into
masterfrom
rdb-js-timeout

Conversation

@russelldb

Copy link
Copy Markdown

js_driver:eval_js/2 uses a macro SCRIPT_TIMEOUT for the amount of
time to wait for a js script to execute before timing out. In some
instances (testing JS map reduce on limited hardware) it would be
handy to set this timeout higher.

Use the 'erlang_js' app_env var 'script_timeout' and any int value.
Timeout is in milliseconds.

js_driver:eval_js/2 uses a macro SCRIPT_TIMEOUT for the amount of
time to wait for a js script to execute before timing out. In some
instances (testing JS map reduce on limited hardware) it would be
handy to set this timeout higher.

Use the 'erlang_js' app_env var 'script_timeout' and any int value.
Timeout is in milliseconds.
@joedevivo

Copy link
Copy Markdown

+1 this worked great in riak 1.2.1 with riak_test, but I did have to rebase before rebar would allow the dependency to work.

@strange

strange commented Jan 28, 2013

Copy link
Copy Markdown

Our use application:get_env/2? A little messier, but nice to get rid of the dep.

@cmeiklejohn

Copy link
Copy Markdown
Contributor

app_helper is only going to work when bundled with riak/riak_core. Should this be modified to use application:get_env as recommended above?

@cmeiklejohn

Copy link
Copy Markdown
Contributor

Closing in favor of #41.

@seancribbs
seancribbs deleted the rdb-js-timeout branch April 1, 2015 22:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants