Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show open and closed indices in _cat/indices #7936

Closed
wants to merge 3 commits into from

Conversation

dadoonet
Copy link
Member

When asking for GET /_cat/indices?v, you can now retrieve closed indices in addition to opened ones.

health status index              pri rep docs.count docs.deleted store.size pri.store.size
yellow open   .marvel-2014.05.21   1   1       8792            0     21.7mb         21.7mb
       close  test
yellow open   .marvel-2014.05.22   1   1       3871            0     10.7mb         10.7mb
red    open   .marvel-2014.05.27   1   1

Closes #7907.

When asking for `GET /_cat/indices?v`, you can now retrieve closed indices in addition to opened ones.

```
health status index              pri rep docs.count docs.deleted store.size pri.store.size
yellow open   .marvel-2014.05.21   1   1       8792            0     21.7mb         21.7mb
       close  test
yellow open   .marvel-2014.05.22   1   1       3871            0     10.7mb         10.7mb
red    open   .marvel-2014.05.27   1   1
```

Closes elastic#7907.

table.startRow();
table.addCell(indexHealth == null ? "red*" : indexHealth.getStatus().toString().toLowerCase(Locale.getDefault()));
table.addCell(state == IndexMetaData.State.OPEN ? (indexHealth == null ? "red*" : indexHealth.getStatus().toString().toLowerCase(Locale.getDefault())) : null);
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe you should use Local.ROOT

@dadoonet
Copy link
Member Author

dadoonet commented Oct 1, 2014

@s1monw PR updated.

@s1monw
Copy link
Contributor

s1monw commented Oct 2, 2014

I think this LGTM - any change we can get a rest test for this?

@s1monw s1monw removed the review label Oct 2, 2014
@dadoonet dadoonet closed this in f0052a5 Oct 3, 2014
dadoonet added a commit that referenced this pull request Oct 3, 2014
When asking for `GET /_cat/indices?v`, you can now retrieve closed indices in addition to opened ones.

```
health status index              pri rep docs.count docs.deleted store.size pri.store.size
yellow open   .marvel-2014.05.21   1   1       8792            0     21.7mb         21.7mb
       close  test
yellow open   .marvel-2014.05.22   1   1       3871            0     10.7mb         10.7mb
red    open   .marvel-2014.05.27   1   1
```

Closes #7907.
Closes #7936.

(cherry picked from commit f0052a5)
dadoonet added a commit that referenced this pull request Oct 3, 2014
When asking for `GET /_cat/indices?v`, you can now retrieve closed indices in addition to opened ones.

```
health status index              pri rep docs.count docs.deleted store.size pri.store.size
yellow open   .marvel-2014.05.21   1   1       8792            0     21.7mb         21.7mb
       close  test
yellow open   .marvel-2014.05.22   1   1       3871            0     10.7mb         10.7mb
red    open   .marvel-2014.05.27   1   1
```

Closes #7907.
Closes #7936.

(cherry picked from commit f0052a5)
@dadoonet dadoonet deleted the pr/7907-cat-close-indices branch October 3, 2014 12:26
@clintongormley clintongormley changed the title Admin: show open and closed indices in _cat/indices Cat API: show open and closed indices in _cat/indices Nov 3, 2014
@michaelsalmon
Copy link

This doesn't seem to have made it into the guide yet.

@clintongormley
Copy link

@dadoonet please could you document this change?

dadoonet added a commit that referenced this pull request Mar 9, 2015
dadoonet added a commit that referenced this pull request Mar 9, 2015
dadoonet added a commit that referenced this pull request Mar 9, 2015
Related to #7936
(cherry picked from commit d9c19cd)
(cherry picked from commit f185f34)
@dadoonet
Copy link
Member Author

dadoonet commented Mar 9, 2015

@michaelsalmon thanks for raising it. I totally missed that.
@clintongormley done in master, 1.x and 1.4 branches.

@michaelsalmon
Copy link

It looks good but I'm pretty sure that line 44 should end -rnk8 now.

@dadoonet
Copy link
Member Author

So true! Will fix. Thanks.

@clintongormley clintongormley added the :Data Management/CAT APIs Text APIs behind /_cat label Mar 19, 2015
clintongormley added a commit that referenced this pull request Apr 4, 2015
Fixed typo in cat indices

Relates to #7936
clintongormley added a commit that referenced this pull request Apr 4, 2015
Fixed typo in cat indices

Relates to #7936
clintongormley added a commit that referenced this pull request Apr 4, 2015
Fixed typo in cat indices

Relates to #7936
clintongormley added a commit that referenced this pull request Apr 4, 2015
Fixed typo in cat indices

Relates to #7936
@clintongormley clintongormley changed the title Cat API: show open and closed indices in _cat/indices Show open and closed indices in _cat/indices Jun 6, 2015
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
Fixed typo in cat indices

Relates to elastic#7936
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
When asking for `GET /_cat/indices?v`, you can now retrieve closed indices in addition to opened ones.

```
health status index              pri rep docs.count docs.deleted store.size pri.store.size
yellow open   .marvel-2014.05.21   1   1       8792            0     21.7mb         21.7mb
       close  test
yellow open   .marvel-2014.05.22   1   1       3871            0     10.7mb         10.7mb
red    open   .marvel-2014.05.27   1   1
```

Closes elastic#7907.
Closes elastic#7936.

(cherry picked from commit f0052a5)
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
Related to elastic#7936
(cherry picked from commit d9c19cd)
(cherry picked from commit f185f34)
mute pushed a commit to mute/elasticsearch that referenced this pull request Jul 29, 2015
Fixed typo in cat indices

Relates to elastic#7936
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.

Show open and closed indices in _cat/indices
4 participants