Skip to content
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

QFS-321 add s3 option to sample setup script #177

Merged
merged 1 commit into from
Aug 9, 2016

Conversation

mckurt
Copy link
Contributor

@mckurt mckurt commented Aug 8, 2016

No description provided.

@@ -121,13 +125,15 @@ main(int argc, char **argv)
}

// Create a simple file with default replication (at most 3)
// or with replication 0 so that it's written to S3
int numReplicas = ( !osFlag ? 3 : 0);
Copy link
Contributor

Choose a reason for hiding this comment

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

osFlag ? 0 : 3

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done.

@codecov-io
Copy link

Current coverage is 75.51% (diff: 100%)

Merging #177 into master will not change coverage

@@             master       #177   diff @@
==========================================
  Files             8          8          
  Lines           245        245          
  Methods          34         34          
  Messages          0          0          
  Branches         31         31          
==========================================
  Hits            185        185          
  Misses           42         42          
  Partials         18         18          

Powered by Codecov. Last update aea4a30...7ea9d5b

@mikeov mikeov merged commit f208637 into quantcast:master Aug 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants