Skip to content

Commit

Permalink
Cherry-pick elastic#5189 to 6.0: cms_get and cmd_set mixed up in memc…
Browse files Browse the repository at this point in the history
…ached module (elastic#5233)

Cherry-pick of PR elastic#5189 to 6.0 branch. Original message: 

I just created this here and thought to look at the code.

https://discuss.elastic.co/t/metricbeat-memcached-module-seems-to-mix-up-memcached-stats-cmd-get-and-memcached-stats-cmd-set/100732
  • Loading branch information
tsg authored and ruflin committed Sep 25, 2017
1 parent e412970 commit 7df2772
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.asciidoc
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,8 @@ https://github.com/elastic/beats/compare/v6.0.0-rc1...master[Check the HEAD diff

*Metricbeat*

- Fixed `cmd_set` and `cmd_get` being mixed in the Memcache module. {pull}5189[5189]

*Packetbeat*

*Winlogbeat*
Expand Down
4 changes: 2 additions & 2 deletions metricbeat/module/memcached/stats/data.go
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ var (
"misses": c.Int("get_misses"),
},
"cmd": s.Object{
"get": c.Int("cmd_set"),
"set": c.Int("cmd_get"),
"get": c.Int("cmd_get"),
"set": c.Int("cmd_set"),
},
"read": s.Object{
"bytes": c.Int("bytes_read"),
Expand Down

0 comments on commit 7df2772

Please sign in to comment.