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

Migrate: Log prospector to the input interface #6120

Merged
merged 1 commit into from
Feb 3, 2018

Conversation

ph
Copy link
Contributor

@ph ph commented Jan 19, 2018

require #6078

@ph ph added in progress Pull request is currently in progress. Filebeat Filebeat labels Jan 19, 2018
@ph ph force-pushed the refactor/log-prospector-to-input branch from a136d42 to bbbbba0 Compare January 19, 2018 21:06
@ph ph changed the title Migrate: Log prospect to the input interace and rename prospector to Migrate: Log prospector to the input interface Jan 19, 2018
@ph ph mentioned this pull request Jan 19, 2018
10 tasks
@ph ph force-pushed the refactor/type-aliasing-input branch from d027f45 to 0e31dc7 Compare January 22, 2018 18:35
@ph ph force-pushed the refactor/log-prospector-to-input branch 2 times, most recently from 4833141 to ece1068 Compare January 23, 2018 02:46
@ph ph changed the base branch from refactor/type-aliasing-input to master January 23, 2018 02:46
@ph ph force-pushed the refactor/log-prospector-to-input branch from ece1068 to 01d7fb2 Compare January 23, 2018 13:44
@ph ph added review and removed in progress Pull request is currently in progress. labels Jan 23, 2018
Copy link
Member

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

@ph LGTM. Left one minor question.

@@ -11,8 +11,8 @@ import (

"github.com/elastic/beats/filebeat/channel"
"github.com/elastic/beats/filebeat/harvester"
input "github.com/elastic/beats/filebeat/input"
Copy link
Member

Choose a reason for hiding this comment

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

Why do you prefix it here with input? It should already be input.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

let me check that, i believe my editor did this change.

@ph ph force-pushed the refactor/log-prospector-to-input branch from 01d7fb2 to d118088 Compare January 30, 2018 21:14
@ph
Copy link
Contributor Author

ph commented Jan 30, 2018

@ruflin rebase and removed the input, not sure about the import, I've never managed the import manually.

@ph ph force-pushed the refactor/log-prospector-to-input branch from d118088 to e0c46be Compare January 31, 2018 14:31
@ph ph force-pushed the refactor/log-prospector-to-input branch from e0c46be to 9b31bd9 Compare February 2, 2018 15:42
@ruflin ruflin merged commit 14f0927 into elastic:master Feb 3, 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.

2 participants