Skip to content
This repository was archived by the owner on Sep 2, 2020. It is now read-only.

Add a basic stack-tracking mechanism for easier debugging.#11

Open
bklimt wants to merge 1 commit into
masterfrom
klimt.stack-tracking
Open

Add a basic stack-tracking mechanism for easier debugging.#11
bklimt wants to merge 1 commit into
masterfrom
klimt.stack-tracking

Conversation

@bklimt

@bklimt bklimt commented May 4, 2014

Copy link
Copy Markdown
Contributor

No description provided.

@bnham

bnham commented May 5, 2014

Copy link
Copy Markdown

I think you could just use a regular "volatile boolean" here rather than an AtomicBoolean. The latter is useful if you need the CAS method, which this commit doesn't seem to use.

@depoll

depoll commented May 6, 2014

Copy link
Copy Markdown

I'd watch out for volatile boolean -- looks like volatile hasn't always been respected by the Dalvik VM (http://stackoverflow.com/questions/4588076/is-dalviks-memory-model-the-same-as-javas)

Comment thread Bolts/src/bolts/Task.java

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could you add a comment about the impact of setting the threadlocal here/what it's intended to do?

@depoll

depoll commented May 6, 2014

Copy link
Copy Markdown

Overall looking pretty good/useful -- just the comments listed above :)

@josephearl

Copy link
Copy Markdown

@bklimt are you intending to update this?

If not I will clone your branch, make the modifications and submit as a new PR.

@grantland

Copy link
Copy Markdown
Member

AFAIK he's not. I've also tried using this internally and I get OOM exceptions.

@josephearl

Copy link
Copy Markdown

Hmm OK, I'll give it a spin and some point see if it's worth continuing.

@grantland

Copy link
Copy Markdown
Member

*internally within our codebase that uses bolts, not just bolts on its own.

@grantland grantland mentioned this pull request Aug 20, 2015
@ghost

ghost commented Jul 12, 2016

Copy link
Copy Markdown

Thank you for your pull request. As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore. In order for us to review and merge your code, please email cla@fb.com with your details so we can update your status.

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.

6 participants