Skip to content

Conversation

@mjsax
Copy link
Member

@mjsax mjsax commented Jan 7, 2016

No description provided.

@StephanEwen
Copy link
Contributor

Can you add a test case that guards this change?

@mjsax
Copy link
Member Author

mjsax commented Jan 8, 2016

Sure. Will do that.

@mjsax
Copy link
Member Author

mjsax commented Jan 10, 2016

Done. Failing Travis is unrelated.

Copy link
Contributor

Choose a reason for hiding this comment

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

-v is a valid option. I would change it to -f which is also the option initially causing the problem.

@fhueske
Copy link
Contributor

fhueske commented Jan 12, 2016

Will change the option to -f and merge this PR

@asfgit asfgit closed this in a5b0556 Jan 12, 2016
asfgit pushed a commit that referenced this pull request Jan 12, 2016
@mjsax mjsax deleted the flink-3189-cli-info branch January 14, 2016 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants