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

Is the example wrong ? #23930

Closed
ZYSzys opened this issue Oct 28, 2018 · 1 comment
Closed

Is the example wrong ? #23930

ZYSzys opened this issue Oct 28, 2018 · 1 comment
Labels
doc Issues and PRs related to the documentations.

Comments

@ZYSzys
Copy link
Member

ZYSzys commented Oct 28, 2018

About the cluster Event:'message',
is this example wrong ?
When I run the code in my machine, It shows numReqs didn't increase as below:

screen shot 2018-10-28 at 8 42 18 am

  • Version: v10.12.0
  • Platform: macOS High Sierra 10.13.3
  • Subsystem:
@Trott
Copy link
Member

Trott commented Oct 28, 2018

The example is correct, I suppose, but in order to cause the number to increase, you need to issue an HTTP request on localhost port 8000. I did this by opening another terminal window and running curl http://localhost:8000.

The example could stand to be either clarified in the text or changed to be more obvious and not require HTTP requests. Or maybe both. But I wouldn't say it's wrong as is. Just could stand some improvement. ¯\(ツ)

@ChALkeR ChALkeR added the doc Issues and PRs related to the documentations. label Oct 28, 2018
@ZYSzys ZYSzys closed this as completed Oct 28, 2018
trivikr pushed a commit that referenced this issue Oct 29, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
targos pushed a commit that referenced this issue Nov 1, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 26, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 26, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
rvagg pushed a commit that referenced this issue Nov 28, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
rvagg pushed a commit that referenced this issue Nov 28, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
MylesBorins pushed a commit that referenced this issue Nov 29, 2018
PR-URL: #23931
Refs: #23930
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations.
Projects
None yet
Development

No branches or pull requests

3 participants