Rebecca Schlussel

Can you explain why you don't just use Character.isJavaIdentifierStart() and Character.isJavaIdentifierPart()? They are constant time operations.

I think this is a duplicate of https://github.com/prestodb/presto-admin/issues/35 It seems like s a valuable addition and the syntax you proposed is good. Right now we special case the `node.id`. Because...

Thanks for the patch! We're excited to have a new contributor! A couple notes on testing: You should run 'make lint' to ensure that the code adheres to our style....

travis failures are real. It looks like you missed a few usages of stop_host

looks good % fixing the test failures.

Can you also revert the timeout increase (and make sure that this fixes the issue)?

@petroav what's the status of this?

So glad you'd like to contribute! We work on pull requests, so to contribute, you should create a fork of this repo, and put your changes there. Then create a...

There are also some configuration properties (e.g. the main presto config directory) that get set directly in the call to the launcher script. We should allow setting those in the...

We have the same issue. I found it was a result of long inode values, which were only introduced in rpm 4.6. (as @rmquapp notes).