Skip to content

add support for RemoteInfo API and adds cross cluster support to Inde…#2743

Merged
Mpdreamz merged 1 commit into5.4from
feature/remote-info
May 4, 2017
Merged

add support for RemoteInfo API and adds cross cluster support to Inde…#2743
Mpdreamz merged 1 commit into5.4from
feature/remote-info

Conversation

@Mpdreamz
Copy link
Copy Markdown
Member

@Mpdreamz Mpdreamz commented May 1, 2017

…xName

@Mpdreamz
Copy link
Copy Markdown
Member Author

Mpdreamz commented May 2, 2017

Technically : is allowed in index names (for now). So this PR would list it as Cluster seperately from Name always on IndexName. However it has no real implications down the line. Also cluster:name:with:semicolons should be parsed correctly although a test for this is missing.

Copy link
Copy Markdown
Contributor

@russcam russcam left a comment

Choose a reason for hiding this comment

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

A couple of minor nitpicks, otherwise LGTM 👍

{
var enableRemoteClusters = client.ClusterPutSettings(new ClusterPutSettingsRequest
{
Transient = new M
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think it's be better to use the type here instead of the M alias, for documentation purposes

public partial interface IElasticClient
{
/// <summary>
/// The cluster health API allows to get a very simple status on the health of the cluster.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should use the remote info API summary?

The cluster remote info API allows to retrieve all of the configured remote cluster information.

private static readonly char[] ClusterSeparator = {':'};
internal string DebugDisplay => Type == null ? Name : $"{nameof(IndexName)} for typeof: {Type?.Name}";

//TODO 6.0 make setters private and use constructor
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

++

@Mpdreamz Mpdreamz merged commit cf63e5c into 5.4 May 4, 2017
Mpdreamz added a commit that referenced this pull request May 4, 2017
@Mpdreamz
Copy link
Copy Markdown
Member Author

Mpdreamz commented May 4, 2017

Implemented PR feedback as part of 265a087 this should be forward ported to master too.

@Mpdreamz Mpdreamz mentioned this pull request May 4, 2017
Mpdreamz added a commit that referenced this pull request May 4, 2017
Mpdreamz added a commit that referenced this pull request May 4, 2017
* Updated api gen to 5.4 and added a way to patch specification files

through special *.patch.json companion files. Due to pending discusion
on
elastic/elasticsearch@e579629
:q!

* updated x-pack spec to 5.4

* add codegen part for xpack info related APIs

* Added support for Field Caps API

* add support for RemoteInfo API and adds cross cluster support to IndexName

* added support for SourceExists()

* add skipversion, eventhough this API existed it was undocumented prior to 5.4

* expose word delimiter graph token filter as per elastic/elasticsearch#23327

* spaces=>tabs

* expose num_reduce_phases as per elastic/elasticsearch#23288

* implemented XPackInfo() started on XPackUsage()

* added response structure for XPackUsage()

* change license date from DateTime to DateTimeOffset'

* implement PR feedback on #2743

* remove explicit folder includes in csproj files
@Mpdreamz
Copy link
Copy Markdown
Member Author

Mpdreamz commented May 4, 2017

ported to master

@Mpdreamz Mpdreamz deleted the feature/remote-info branch May 4, 2017 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants