mirror of
https://github.com/minio/minio.git
synced 2024-12-27 23:55:56 -05:00
a2a8d54bb6
This change adds `access` format support for notifications to a Elasticsearch server, and it refactors `namespace` format support. In the case of `access` format, for each event in Minio, a JSON document is inserted into Elasticsearch with its timestamp set to the event's timestamp, and with the ID generated automatically by elasticsearch. No events are modified or deleted in this mode. In the case of `namespace` format, for each event in Minio, a JSON document is keyed together by the bucket and object name is updated in Elasticsearch. In the case of an object being created or over-written in Minio, a new document or an existing document is inserted into the Elasticsearch index. If an object is deleted in Minio, the corresponding document is deleted from the Elasticsearch index. Additionally, this change upgrades Elasticsearch support to the 5.x series. This is a breaking change, and users of previous elasticsearch versions should upgrade. Also updates documentation on Elasticsearch notification target usage and has a link to an elasticsearch upgrade guide. This is the last patch that finally resolves #3928.
41 lines
1.9 KiB
Markdown
41 lines
1.9 KiB
Markdown
# How to contribute
|
|
|
|
Elastic is an open-source project and we are looking forward to each
|
|
contribution.
|
|
|
|
Notice that while the [official Elasticsearch documentation](https://www.elastic.co/guide/en/elasticsearch/reference/current/index.html) is rather good, it is a high-level
|
|
overview of the features of Elasticsearch. However, Elastic tries to resemble
|
|
the Java API of Elasticsearch which you can find [on GitHub](https://github.com/elastic/elasticsearch).
|
|
|
|
This explains why you might think that some options are strange or missing
|
|
in Elastic, while often they're just different. Please check the Java API first.
|
|
|
|
Having said that: Elasticsearch is moving fast and it might be very likely
|
|
that we missed some features or changes. Feel free to change that.
|
|
|
|
## Your Pull Request
|
|
|
|
To make it easy to review and understand your changes, please keep the
|
|
following things in mind before submitting your pull request:
|
|
|
|
* You compared the existing implemenation with the Java API, did you?
|
|
* Please work on the latest possible state of `olivere/elastic`.
|
|
Use `release-branch.v2` for targeting Elasticsearch 1.x and
|
|
`release-branch.v3` for targeting 2.x.
|
|
* Create a branch dedicated to your change.
|
|
* If possible, write a test case which confirms your change.
|
|
* Make sure your changes and your tests work with all recent versions of
|
|
Elasticsearch. We currently support Elasticsearch 1.7.x in the
|
|
release-branch.v2 and Elasticsearch 2.x in the release-branch.v3.
|
|
* Test your changes before creating a pull request (`go test ./...`).
|
|
* Don't mix several features or bug fixes in one pull request.
|
|
* Create a meaningful commit message.
|
|
* Explain your change, e.g. provide a link to the issue you are fixing and
|
|
probably a link to the Elasticsearch documentation and/or source code.
|
|
* Format your source with `go fmt`.
|
|
|
|
## Additional Resources
|
|
|
|
* [GitHub documentation](http://help.github.com/)
|
|
* [GitHub pull request documentation](http://help.github.com/send-pull-requests/)
|