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

_count requests broken as a of commit 7bf0f1ffca589df6e626d61182689bde005ce649 #52

Closed
clintongormley opened this Issue Mar 8, 2010 · 4 comments

Comments

Projects
None yet
2 participants
@clintongormley
Member

clintongormley commented Mar 8, 2010

The server now hangs with _count queries. A bisect shows:

7bf0f1f is the first bad commit
commit 7bf0f1f
Author: kimchy kimchy@gmail.com
Date: Fri Mar 5 01:39:04 2010 +0200

refactor client api, remove execXXX, and simple remain with the actual operation name as the method name, one that returns a future, and one that accepts a listener

:040000 040000 941417e6df82b84dbd3ac28576aa589916d398ea c2c27160d53974c1c78a3b9108ce8f3ca16c665e M modules

Test script:

curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/1'  -d '
{
   "num" : 2,
   "text" : "foo"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/2'  -d '
{
   "num" : 3,
   "text" : "foo"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/3'  -d '
{
   "num" : 4,
   "text" : "foo"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/4'  -d '
{
   "num" : 5,
   "text" : "foo"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/5'  -d '
{
   "num" : 6,
   "text" : "foo bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/6'  -d '
{
   "num" : 7,
   "text" : "foo bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/7'  -d '
{
   "num" : 8,
   "text" : "foo bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/8'  -d '
{
   "num" : 9,
   "text" : "foo bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/9'  -d '
{
   "num" : 10,
   "text" : "foo bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/10'  -d '
{
   "num" : 11,
   "text" : "foo bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/11'  -d '
{
   "num" : 12,
   "text" : "foo bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/12'  -d '
{
   "num" : 13,
   "text" : "foo bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/13'  -d '
{
   "num" : 14,
   "text" : "bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/14'  -d '
{
   "num" : 15,
   "text" : "bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/15'  -d '
{
   "num" : 16,
   "text" : "bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/16'  -d '
{
   "num" : 17,
   "text" : "bar baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/17'  -d '
{
   "num" : 18,
   "text" : "baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/18'  -d '
{
   "num" : 19,
   "text" : "baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/19'  -d '
{
   "num" : 20,
   "text" : "baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/20'  -d '
{
   "num" : 21,
   "text" : "baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/21'  -d '
{
   "num" : 22,
   "text" : "bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/22'  -d '
{
   "num" : 23,
   "text" : "bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/23'  -d '
{
   "num" : 24,
   "text" : "bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/24'  -d '
{
   "num" : 25,
   "text" : "bar"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_1/25'  -d '
{
   "num" : 26,
   "text" : "foo baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_1/type_2/26'  -d '
{
   "num" : 27,
   "text" : "foo baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_1/27'  -d '
{
   "num" : 28,
   "text" : "foo baz"
}
'
curl -XPUT 'http://127.0.0.2:9200/es_test_2/type_2/28'  -d '
{
   "num" : 29,
   "text" : "foo baz"
}
'
curl -XGET 'http://127.0.0.2:9200/_cluster/health?timeout=2&waitForStatus=green' 
curl -XGET 'http://127.0.0.2:9200/_all/_count'  -d '
{
   "term" : {
      "text" : "foo"
   }
}
'
@kimchy

This comment has been minimized.

Show comment
Hide comment
@kimchy

kimchy Mar 9, 2010

Member

_count hangs, closed by 83a78b3

Member

kimchy commented Mar 9, 2010

_count hangs, closed by 83a78b3

@kimchy

This comment has been minimized.

Show comment
Hide comment
@kimchy

kimchy Mar 9, 2010

Member

Fixed, my bad!. By the way, int the health API, you specify a timeout of 2, which is 2 milliseconds..., you should set it to something like 2s (so its 2 seconds).

Member

kimchy commented Mar 9, 2010

Fixed, my bad!. By the way, int the health API, you specify a timeout of 2, which is 2 milliseconds..., you should set it to something like 2s (so its 2 seconds).

@kimchy

This comment has been minimized.

Show comment
Hide comment
@kimchy

kimchy Mar 9, 2010

Member

Releasing 0.5.1 to fix this...

Member

kimchy commented Mar 9, 2010

Releasing 0.5.1 to fix this...

@clintongormley

This comment has been minimized.

Show comment
Hide comment
@clintongormley

clintongormley Mar 9, 2010

Member

Fix confirmed. And thanks for the note about the timeout

Member

clintongormley commented Mar 9, 2010

Fix confirmed. And thanks for the note about the timeout

dadoonet added a commit that referenced this issue Jun 5, 2015

dadoonet added a commit that referenced this issue Jun 5, 2015

[cleanup] Change Logger factory
Related to #52

(cherry picked from commit a6435ab)
(cherry picked from commit f82ee58)

dadoonet added a commit that referenced this issue Jun 5, 2015

[cleanup] simplify logging debug/trace code
[logging] don't use anymore `if (logger.isTraceEnabled())` or `if (logger.isDebugEnabled())`

Related to #52.

(cherry picked from commit 95381d4)
(cherry picked from commit 6d5ce44)

dadoonet added a commit that referenced this issue Jun 5, 2015

[cleanup] remove useless `;`
Related to #52.

(cherry picked from commit 0899438)
(cherry picked from commit d1a9b0b)

dadoonet added a commit that referenced this issue Jun 5, 2015

[cleanup] use diamond operator
With Java 7, we can change `Set<Instance> = new HashSet<Instance>()` to `Set<Instance> = new HashSet<>()`

Related to #52.

(cherry picked from commit 0c709de)
(cherry picked from commit 4f474f5)

dadoonet added a commit that referenced this issue Jun 5, 2015

[cleanup] collapse identical catch blocks
With Java7, you don't need multiple identical catch blocks anymore

Related to #52.

(cherry picked from commit 322e1e5)
(cherry picked from commit 9d3b0ad)

StefanGor pushed a commit to StefanGor/elasticsearch that referenced this issue Mar 17, 2017

ywelsch pushed a commit to ywelsch/elasticsearch that referenced this issue May 24, 2018

Improve support for iterated LegislatorTests (#58)
The Legislator tests are currently a little painful to run in an iterated
fashion (i.e. with `-Dtests.iters=BIGNUM`) due to the quantity of log output
produced. This change removes the TRACE-by-default annotations, and also
reduces the number of iterations in `runRandomly()`.

Verbose logging can be reinstated by running the tests with

    -Dtests.es.logger.level=TRACE

Fixes #52

This issue was closed.

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