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

Remove network stats & info #12054

Merged
merged 1 commit into from Jul 7, 2015
Merged

Remove network stats & info #12054

merged 1 commit into from Jul 7, 2015

Conversation

tlrx
Copy link
Member

@tlrx tlrx commented Jul 6, 2015

This pull request removes Network stats and Network info.

The following information are removed from network section of the Node Stats API:

  • network.tcp
  • network.tcp.active_opens
  • network.tcp.passive_opens
  • network.tcp.curr_estab
  • network.tcp.in_segs
  • network.tcp.out_segs
  • network.tcp.retrans_segs
  • network.tcp.estab_resets
  • network.tcp.attempt_fails
  • network.tcp.in_errs
  • network.tcp.out_rsts

The following information are removed from network section of the Node Info API:

  • network.refresh_interval_in_millis
  • network.primary_interface
  • network.primary_interface.address
  • network.primary_interface.name
  • network.primary_interface.mac_address

@tlrx tlrx added >breaking v2.0.0-beta1 review :Data Management/Stats Statistics tracking and retrieval APIs labels Jul 6, 2015
@tlrx
Copy link
Member Author

tlrx commented Jul 6, 2015

@kimchy Can you have a look please? thanks

@kimchy
Copy link
Member

kimchy commented Jul 7, 2015

LGTM

@tlrx tlrx merged commit 30892c4 into elastic:master Jul 7, 2015
@kevinkluge kevinkluge removed the review label Jul 7, 2015
@tlrx tlrx deleted the remove-network-stats branch July 8, 2015 13:18
@tlrx tlrx mentioned this pull request Jul 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>breaking :Data Management/Stats Statistics tracking and retrieval APIs v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants