Skip to content

Comments

[DOCS] Update the number detection mapping rules for float#66911

Closed
RickyLau wants to merge 1 commit intoelastic:masterfrom
ElasticsearchClub:master
Closed

[DOCS] Update the number detection mapping rules for float#66911
RickyLau wants to merge 1 commit intoelastic:masterfrom
ElasticsearchClub:master

Conversation

@RickyLau
Copy link
Contributor

@RickyLau RickyLau commented Jan 4, 2021

Update the number detection mapping rules for floating-point values in JSON

Closes #66910

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Jan 4, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

Thanks @RickyLau .

@javanna javanna changed the title [DOCS] Update the number detection mapping rules for floating-point v… [DOCS] Update the number detection mapping rules for float Jan 5, 2021
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Jan 5, 2021
@javanna
Copy link
Contributor

javanna commented Jan 7, 2021

@RickyLau are you planning to accept the suggested change? If so we can then merge your change. Thanks!

@RickyLau
Copy link
Contributor Author

RickyLau commented Jan 9, 2021

@RickyLau are you planning to accept the suggested change? If so we can then merge your change. Thanks!

I accept this suggestion.

@javanna
Copy link
Contributor

javanna commented Jan 11, 2021

thanks @RickyLau , could you go and accept the suggestion above so the change becomes a commit and we can merge it in? Unfortunately a comment is not enough to make it a commit ;)

lockewritesdocs pushed a commit that referenced this pull request Jan 11, 2021
* Changes for dynamic templates.

* Clarifying language around dynamic:true and dynamic:runtime.

* Clarifying edits and some restructuring.

* Overhauling the Mapping page.

* Incorporating changes from #66911.

* Reworking mapping page to focus on dynamic vs. explicit mapping.

* Reordering to fix test failure.

* Further clarifying mapping page.

* Reordering sections, adding headings to examples, and other clarifications.

* Incorporating review feedback.

* Adding description of  for Painless script.
lockewritesdocs pushed a commit that referenced this pull request Jan 11, 2021
* Changes for dynamic templates.

* Clarifying language around dynamic:true and dynamic:runtime.

* Clarifying edits and some restructuring.

* Overhauling the Mapping page.

* Incorporating changes from #66911.

* Reworking mapping page to focus on dynamic vs. explicit mapping.

* Reordering to fix test failure.

* Further clarifying mapping page.

* Reordering sections, adding headings to examples, and other clarifications.

* Incorporating review feedback.

* Adding description of  for Painless script.

* Minor revisions.
lockewritesdocs pushed a commit that referenced this pull request Jan 11, 2021
* Changes for dynamic templates.

* Clarifying language around dynamic:true and dynamic:runtime.

* Clarifying edits and some restructuring.

* Overhauling the Mapping page.

* Incorporating changes from #66911.

* Reworking mapping page to focus on dynamic vs. explicit mapping.

* Reordering to fix test failure.

* Further clarifying mapping page.

* Reordering sections, adding headings to examples, and other clarifications.

* Incorporating review feedback.

* Adding description of  for Painless script.

* Minor revisions.
@jrodewig
Copy link
Contributor

#66530 has implemented this change. I'm going to close this PR. If I've missed something, please feel free to re-open. Thanks for raising this @RickyLau!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes Team:Docs Meta label for docs team v7.11.0 v7.12.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error documentation for dynamic field mapping with JSON string type

6 participants