Skip to content

Adding info about Elasticsearch#337

Open
volym3ad wants to merge 3 commits into
opsschool:masterfrom
volym3ad:add-elasticsearch-info
Open

Adding info about Elasticsearch#337
volym3ad wants to merge 3 commits into
opsschool:masterfrom
volym3ad:add-elasticsearch-info

Conversation

@volym3ad

@volym3ad volym3ad commented Mar 6, 2018

Copy link
Copy Markdown

No description provided.

@miketheman miketheman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice stuff for a first pass! I've made a few structural comments, and would recommend that @avleen review the content further, as he's a bit deeper steeped in Elasticsearch than I.

Comment thread databases_201.rst Outdated
Comment thread databases_201.rst Outdated
Comment thread databases_201.rst Outdated
Comment thread databases_201.rst Outdated
Comment thread databases_201.rst Outdated
@volym3ad

volym3ad commented Mar 7, 2018

Copy link
Copy Markdown
Author

yeah, I'm a newbie to rst... I'll fix asap.

@volym3ad

Copy link
Copy Markdown
Author

updated rst docs according to required changes.

@avleen @miketheman can you please review?

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.

3 participants