Skip to content

Conversation

@xescugc
Copy link
Contributor

@xescugc xescugc commented Dec 16, 2016

The catalan analyzer was repeated on the supported list

The `catalan` analyzer was repeated on the supported list :)
@xescugc
Copy link
Contributor Author

xescugc commented Dec 16, 2016

I have the documents signed and the emails of confirmation 😢

@clintongormley
Copy link
Contributor

Hi @xescugc

thanks for the PR. Could you remove the other catalan instead? That way the list is in alphabetic order

thanks

@clintongormley clintongormley added the >docs General docs changes label Dec 19, 2016
@xescugc
Copy link
Contributor Author

xescugc commented Dec 19, 2016

I've changed the order for this one (catalan), if you want I can order them all because they are not hehe (https://github.com/elastic/elasticsearch/pull/22240/files#diff-5cedbea057491ac3dac21fc11f2ec575R59)

@clintongormley
Copy link
Contributor

if you want I can order them all because they are not

that would be great. also note that finnish appears twice too :)

@xescugc
Copy link
Contributor Author

xescugc commented Dec 20, 2016

Removed the repeated finnish and reordered the languages 😄

@clintongormley clintongormley merged commit 2e0f70c into elastic:5.1 Dec 21, 2016
@clintongormley
Copy link
Contributor

thanks @xescugc - merged

clintongormley pushed a commit that referenced this pull request Dec 21, 2016
* Repeated language analyzers

The `catalan` analyzer was repeated on the supported list :)

* Reordered the languages to have alphabetic order

* Added space for format

* Reordered the languages and removed repeated
clintongormley pushed a commit that referenced this pull request Dec 21, 2016
* Repeated language analyzers

The `catalan` analyzer was repeated on the supported list :)

* Reordered the languages to have alphabetic order

* Added space for format

* Reordered the languages and removed repeated
clintongormley pushed a commit that referenced this pull request Dec 21, 2016
* Repeated language analyzers

The `catalan` analyzer was repeated on the supported list :)

* Reordered the languages to have alphabetic order

* Added space for format

* Reordered the languages and removed repeated
@xescugc
Copy link
Contributor Author

xescugc commented Dec 22, 2016

Was a pleasure to help 😄

@xescugc xescugc deleted the patch-1 branch December 22, 2016 14:29
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Dec 22, 2016
* master: (22 commits)
  Support negative numbers in writeVLong (elastic#22314)
  UnicastZenPing's PingingRound should prevent opening connections after being closed
  Add task to clean idea build directory. Make cleanIdea task invoke it.
  add trace logging to UnicastZenPingTests.testResolveReuseExistingNodeConnections
  Adds ingest processor headers to exception for unknown processor. (elastic#22315)
  Remove much ceremony from parsing client yaml test suites (elastic#22311)
  Support numeric bounds with decimal parts for long/integer/short/byte datatypes (elastic#21972)
  inner hits: Don't inline inner hits if the query the inner hits is inlined into can't resolve mappings and ignore_unmapped has been set to true
  Fix stackoverflow error on InternalNumericMetricAggregation
  Date detection should not rely on a hardcoded set of characters. (elastic#22171)
  `value_type` is useful regardless of scripting. (elastic#22160)
  Improve concurrency of ShardCoreKeyMap. (elastic#22316)
  fixed jdocs and removed already fixed norelease
  Adds abstract test classes for serialisation (elastic#22281)
  Introduce translog no-op
  Provide helpful error message if a plugin exists
  Clear static variable after suite
  Repeated language analyzers (elastic#22240)
  Restore deprecation warning for invalid match_mapping_type values (elastic#22304)
  Make `-0` compare less than `+0` consistently. (elastic#22173)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>docs General docs changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants