Comments (7)
this ruby code in a file that is loaded with LoadFile will break it:
sleep 600
from go-mruby.
@erikh Do you have exact code & steps we can use to reproduce this?
I pulled your load file patch and tried with implementations of sleep
using both golang and mruby-sleep mgem but could not reproduce. Is there any chance that other goroutines could be sharing the mruby interpreter?
I did find this which may be relevant.
from go-mruby.
let me see if I can find the test case I used, but IIRC, I converted
github.com/erikh/box to use load_file instead of load_string (which is what
already exists). this is what caused the behavior.
box's tests build for about 20 minutes now, so it tripped in there
somewhere.
I'll try to investigate soon. If you are feeling confident in the patch,
I'm not going to object to it being merged if my case was an edge one.
On Fri, Nov 11, 2016 at 5:16 AM, Mike Simons [email protected]
wrote:
@erikh https://github.com/erikh Do you have exact code & steps we can
use to reproduce this?I pulled your load file patch and tried with implementations of sleep
using both golang and mruby-sleep mgem but could not reproduce. Is there
any chance that other goroutines could be sharing the mruby interpreter?I did find this
golang/go#15639 (comment) which
may be relevant.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ6w8cJFKjZurvW7HPEAKSB6FqmP2iks5q9GqYgaJpZM4KjGyY
.
from go-mruby.
My gut feel is that this is an issue albeit one that is triggered with a very specific, possibly compound set of steps. I haven't been able to replicate it though, even with your patch applied to box (and modified to use LoadFile).
This is the kind of tricky issue that is going to crop up and put folks off using it for stability concerns so I think it is important to understand the reasons behind it so we can either mitigate or inform potential users.
from go-mruby.
Sorry I was a little short, basically, what I did was make a long running
open mrb_load_file (running ruby program), at which point I got this error.
maybe a platform issue?
On Thu, Nov 17, 2016 at 4:44 AM, Mike Simons [email protected]
wrote:
My gut feel is that this is an issue albeit one that is triggered with
a very specific, possibly compound set of steps. I haven't been able to
replicate it though, even with your patch applied to box (and modified to
use LoadFile).This is the kind of tricky issue that is going to crop up and put folks
off using it for stability concerns so I think it is important to
understand the reasons behind it so we can either mitigate or inform
potential users.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ67hW3kRkvntr5BNJIwSx_SjDCQvDks5q_EwRgaJpZM4KjGyY
.
from go-mruby.
for now, the code is reverted, I'm hoping I can resolve this soon.
On Thu, Nov 17, 2016 at 8:28 AM, Erik Hollensbe [email protected] wrote:
Sorry I was a little short, basically, what I did was make a long running
open mrb_load_file (running ruby program), at which point I got this error.
maybe a platform issue?On Thu, Nov 17, 2016 at 4:44 AM, Mike Simons [email protected]
wrote:My gut feel is that this is an issue albeit one that is triggered with
a very specific, possibly compound set of steps. I haven't been able to
replicate it though, even with your patch applied to box (and modified to
use LoadFile).This is the kind of tricky issue that is going to crop up and put folks
off using it for stability concerns so I think it is important to
understand the reasons behind it so we can either mitigate or inform
potential users.—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#40 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJ67hW3kRkvntr5BNJIwSx_SjDCQvDks5q_EwRgaJpZM4KjGyY
.
from go-mruby.
@mikesimons ar eyou still using this project? do you still see this on recent versions? I have run a lot of builds through box now -- almost 10k/day now -- and I never see it.
from go-mruby.
Related Issues (20)
- mruby.Run does not work with a Proc returned from LoadString HOT 6
- mruby.Hash / mruby.Array may not behave as ruby users expect
- Passing large string back to mruby HOT 6
- Array is missing methods for mutating it HOT 2
- It's not clear from the docs how different types should be initialised from Go
- fatal error: exitsyscall: syscall frame is no longer valid HOT 18
- There is no license or copying file HOT 2
- Readme missing dependencies HOT 3
- Can't tell from the docs but is there an Encode which is the pair of Decode HOT 7
- How can I get `Hash` value via a `Symbol` key? HOT 1
- Concurreny Support HOT 4
- Tests doesn't pass on go1.10 HOT 2
- Execute compiled code
- Help me understand LoadString, unexpected behavior HOT 3
- mruby 1.4.1 dispatch breaks go-mruby
- Build fails with: could not determine kind of name for C.mrb_ary_len HOT 7
- golint moved things again
- Ruby calling Go code
- State of this Project? mruby 3.0.0? HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from go-mruby.