New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ACCUMULO-4572 Avoid blocking when looking for files w/o sample #203

Merged
merged 1 commit into from Jan 24, 2017

Conversation

Projects
None yet
3 participants
@keith-turner
Contributor

keith-turner commented Jan 20, 2017

No description provided.

@joshelser

This comment has been minimized.

Show comment
Hide comment
@joshelser

joshelser Jan 20, 2017

Member

Looks OK to me. Any thoughts on a unit or integration test?

Member

joshelser commented Jan 20, 2017

Looks OK to me. Any thoughts on a unit or integration test?

@keith-turner

This comment has been minimized.

Show comment
Hide comment
@keith-turner

keith-turner Jan 20, 2017

Contributor

Looks OK to me. Any thoughts on a unit or integration test?

I think its well covered by existing test. ShellServerIT exercises this code from the shell.

Contributor

keith-turner commented Jan 20, 2017

Looks OK to me. Any thoughts on a unit or integration test?

I think its well covered by existing test. ShellServerIT exercises this code from the shell.

@asfgit asfgit merged commit 95be9f3 into apache:1.8 Jan 24, 2017

2 checks passed

Jenkins This pull request looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment