Update numbers to reflect 4-byte UTF-8-encoded characters #27083
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
|
@elasticmachine please test this again |
jasontedor
added a commit
to jasontedor/elasticsearch
that referenced
this pull request
Oct 24, 2017
* master: Timed runnable should delegate to abstract runnable Expose adaptive replica selection stats in /_nodes/stats API Remove dangerous `ByteBufStreamInput` methods (elastic#27076) Blacklist Gradle 4.2 and above Remove duplicated test (elastic#27091) Update numbers to reflect 4-byte UTF-8-encoded characters (elastic#27083) test: avoid generating duplicate multiple fields (elastic#27080) Reduce the default number of cached queries. (elastic#26949) removed unused import ingest: date processor should not fail if timestamp is specified as json number
|
@DaveCTurner The fix for this PR is available at https://www.elastic.co/guide/en/elasticsearch/reference/master/ignore-above.html, but not in 5.x or 6.x documentation. Can it be backported? |
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
DaveCTurner
added a commit
that referenced
this pull request
Jul 2, 2018
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
|
Yes, apologies @inqueue, this looks like it should have been backported everywhere. Now done. |
dnhatn
added a commit
that referenced
this pull request
Jul 4, 2018
* 6.x: Fix not waiting for Netty ThreadDeathWatcher in IT (#31758) (#31789) [Docs] Correct default window_size (#31582) S3 fixture should report 404 on unknown bucket (#31782) [ML] Limit ML filter items to 10K (#31731) Fixture for Minio testing (#31688) [ML] Return statistics about forecasts as part of the jobsstats and usage API (#31647) [DOCS] Add missing get mappings docs to HLRC (#31765) [DOCS] Starting Elasticsearch (#31701) Fix coerce validation_method in GeoBoundingBoxQueryBuilder (#31747) Painless: Complete Removal of Painless Type (#31699) Consolidate watcher setting update registration (#31762) [DOCS] Adds empty 6.3.1 release notes page ingest: Introduction of a bytes processor (#31733) [test] don't run bats tests for suse boxes (#31749) Add analyze API to high-level rest client (#31577) Implemented XContent serialisation for GetIndexResponse (#31675) [DOCS] Typos DOC: Add examples to the SQL docs (#31633) Add support for AWS session tokens (#30414) Watcher: Reenable start/stop yaml tests (#31754) JDBC: Fix stackoverflow on getObject and timestamp conversion (#31735) Support multiple system store types (#31650) Add write*Blob option to replace existing blob (#31729) Split CircuitBreaker-related tests (#31659) Painless: Add Context Docs (#31190) Docs: Remove missing reference Migrate scripted metric aggregation scripts to ScriptContext design (#30111) Watcher: Fix chain input toXcontent serialization (#31721) Remove _all example (#31711) rest-high-level: added get cluster settings (#31706) Docs: Match the examples in the description (#31710) [Docs] Correct typos (#31720) Extend allowed characters for grok field names (#21745) (#31653) (#31722) [DOCS] Check for Windows and *nix file paths (#31648) [ML] Validate ML filter_id (#31535) Fix gradle4.8 deprecation warnings (#31654) Update numbers to reflect 4-byte UTF-8-encoded characters (#27083)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
You need 4 bytes for characters outside the BMP, which includes many emoji and a bunch of less-common writing characters too.
Fixes #27060.