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

This brings back --filter-workers for the 2.x series. This will be go… #4672

Closed

Conversation

andrewvc
Copy link
Contributor

…ne in the next major version


if filter_workers
@logger.warn("--filter-workers is deprecated! Please use --pipeline-workers. This setting will be removed in the next major version!")
self.pipeline_workers=filter_workers
Copy link
Contributor

Choose a reason for hiding this comment

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

spacing around =

@jordansissel
Copy link
Contributor

Can you update the commit message to say Fixes #4669 on a line by itself at the end?

@@ -134,6 +138,12 @@ def execute
end
configure


if filter_workers
@logger.warn("--filter-workers is deprecated! Please use --pipeline-workers. This setting will be removed in the next major version!")
Copy link
Contributor

Choose a reason for hiding this comment

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

Add

Please use --pipeline-workers or -w. This setting..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

can do.

@andrewvc andrewvc force-pushed the fix_missing_pipeline_Workers_flag branch from 76a46d1 to 4eaa30a Compare February 10, 2016 22:01
@andrewvc andrewvc force-pushed the fix_missing_pipeline_Workers_flag branch from 4eaa30a to 86dbfd6 Compare February 10, 2016 22:02
@andrewvc
Copy link
Contributor Author

@suyograo @jordansissel I believe I have addressed all concerns here with my last force push.

@suyograo
Copy link
Contributor

LGTM

@elasticsearch-bot
Copy link

Andrew Cholakian merged this into the following branches!

Branch Commits
2.2 cf31ef8
2.x 8c9aa4e

elasticsearch-bot pushed a commit that referenced this pull request Feb 10, 2016
elasticsearch-bot pushed a commit that referenced this pull request Feb 10, 2016
@suyograo suyograo closed this Feb 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants