-
Notifications
You must be signed in to change notification settings - Fork 113
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
Updating README and local config #4211
Open
sethstoudenmier
wants to merge
12
commits into
qat
Choose a base branch
from
update-readme-steps
base: qat
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
a31ebe3
Updating README and local config
sethstoudenmier 42cce41
commenting out bind mount to avoid conflict with travis
sethstoudenmier a800f24
removing initdb for conflict with travis
sethstoudenmier 8a0e990
add ES_HOSTNAME to bulk download container
sethstoudenmier a79c89f
fixing test
sethstoudenmier 6603604
update to use JDK 8
sethstoudenmier 3dcf53b
cleanup
sethstoudenmier 979344c
update Dockerfile.spark
sethstoudenmier 732901e
Merge branch 'qat' of github.com:fedspendingtransparency/usaspending-…
sethstoudenmier e26657d
Merge branch 'qat' into update-readme-steps
sethstoudenmier 728f5d0
add JAVA_HOME
sethstoudenmier 5be23e7
Merge branch 'update-readme-steps' of github.com:fedspendingtranspare…
sethstoudenmier File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,7 +18,7 @@ tmp/* | |
spark-warehouse/ | ||
derby.log | ||
|
||
data/output/ | ||
data/* | ||
|
||
# pyenv ignores | ||
\.python-version | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we remove the - between docker compose? I have historically always used docker-compose. Is there a difference? For example:
docker-compose up usaspending-db usaspending-es
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayubshahab Recent updates to
docker
have addedcompose
as a sub-command ofdocker
itself instead of being a standalone executable.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ayubshahab here is the doc to help explain: https://docs.docker.com/compose/releases/migrate/