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

[7.x] Journalbeat: add index option to input (#15071) #15131

Merged
merged 1 commit into from
Dec 17, 2019

Conversation

ycombinator
Copy link
Contributor

Backports the following commits to 7.x:

* Refactoring: renaming var to avoid collision

* Refactoring: extract function

* Refactoring: moving AddFormattedIndex processor into libbeat

* Add constructor for addFormattedIndex processor

* Export processor struct

* Add comment for exported constructor

* Adding comment for exported method

* Adding index option to doc

* Refactoring: extracting common code

* Adding unit tests

* Adding godoc for exported methods
// specific language governing permissions and limitations
// under the License.

package add_formatted_index

Choose a reason for hiding this comment

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

don't use an underscore in package name

Copy link
Contributor

@faec faec left a comment

Choose a reason for hiding this comment

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

Don't forget to add the changelog!

otherwise lgtm

@ycombinator
Copy link
Contributor Author

Yeah, CHANGELOG PR to master is up, just waiting for at least a green Travis CI: #15104. Once that's merged, I'll backport it.

@ycombinator
Copy link
Contributor Author

ycombinator commented Dec 17, 2019

CI test failures in x-pack/metricbeat are unrelated to this PR and also happening on 7.x. Merging.

@ycombinator ycombinator merged commit 88fa2d7 into elastic:7.x Dec 17, 2019
@ycombinator ycombinator deleted the backport/7.x/pr-15071 branch December 17, 2019 00:24
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