Skip to content

Add Filebeat module to state API#7532

Merged
exekias merged 1 commit intoelastic:masterfrom
ruflin:module-state
Jul 9, 2018
Merged

Add Filebeat module to state API#7532
exekias merged 1 commit intoelastic:masterfrom
ruflin:module-state

Conversation

@ruflin
Copy link
Copy Markdown
Contributor

@ruflin ruflin commented Jul 6, 2018

The document looks as following:

{
  "module": {
    "count": 1,
    "names": ["redis"]
  }
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should omit 2nd value from range; this loop is equivalent to for m := range ...

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should omit 2nd value from range; this loop is equivalent to for m := range ...

@ruflin ruflin mentioned this pull request Jul 6, 2018
9 tasks
@ruflin
Copy link
Copy Markdown
Contributor Author

ruflin commented Jul 9, 2018

I created #7536 for the refactoring.

@ruflin ruflin force-pushed the module-state branch 2 times, most recently from 0e18aae to 9eb3f43 Compare July 9, 2018 09:35
The document looks as following:

```
{
  "module": {
    "count": 1,
    "names": ["redis"]
  }
}
```
@exekias exekias merged commit 78d2f33 into elastic:master Jul 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants