Skip to content
New issue

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

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

Already on GitHub? # to your account

Updated README.md #1980

Merged
merged 1 commit into from
Aug 16, 2016
Merged

Updated README.md #1980

merged 1 commit into from
Aug 16, 2016

Conversation

abhijith0505
Copy link
Contributor

Changed the order of the Start Pinpoint Daemons from

Collector - Run quickstart/bin/start-collector.sh

Web UI - Run quickstart/bin/start-web.sh

TestApp - Run quickstart/bin/start-testapp.sh


TO:

Collector - Run quickstart/bin/start-collector.sh

TestApp - Run quickstart/bin/start-testapp.sh

Web UI - Run quickstart/bin/start-web.sh

Which is the correct order

Changed the order of the *Start Pinpoint Daemons* from

**Collector** - Run `quickstart/bin/start-collector.sh`

**Web UI** - Run `quickstart/bin/start-web.sh`

**TestApp** - Run `quickstart/bin/start-testapp.sh`

_______________________________
TO:


**Collector** - Run `quickstart/bin/start-collector.sh`

**TestApp** - Run `quickstart/bin/start-testapp.sh`

**Web UI** - Run `quickstart/bin/start-web.sh`


Which is the correct order
@Xylus
Copy link
Contributor

Xylus commented Aug 16, 2016

@abhijith0505
Thanks for the fix.
Although Collector -> Web -> TestApp should work, your fix is more elegant as it would eliminate the possibility of the Web UI having an empty list of applications which might lead to confusion.
Thanks!

@Xylus Xylus merged commit 958fc94 into pinpoint-apm:master Aug 16, 2016
koo-taejin added a commit that referenced this pull request Aug 16, 2016
updated related document.
koo-taejin added a commit that referenced this pull request Aug 16, 2016
updated related document.
@abhijith0505
Copy link
Contributor Author

@Xylus
Pinpoint is a great open source APM. Wonderful to have it here!
Kudos to the team!

@nstopkimsk
Copy link
Contributor

@abhijith0505 thanks.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants