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

Aggregations: DateHistogram with negative 'pre_offset' or 'post_offset' value ends with "Message not fully read (response) for" #7312

Closed
marcelog opened this issue Aug 18, 2014 · 1 comment · Fixed by #7313

Comments

@marcelog
Copy link

Hey guys,

We're using version 1.3.2 and Oracle JRE: 1.7.0_67. The cluster has 3 nodes (2 data nodes and 1 used only to route searches).

There is a separate (different named) cluster of 1 node using marvel to collect statistics from the first one. No other nodes in the network, and no custom/manually made java clients around.

As the title says, when doing a DateHistogramAggregation using a negative value for either pre_offset or post_offset yields the message:

" [transport.netty ] Message not fully read (response) for [69208] handler org.elasticsearch.search.action.SearchServiceTransportAction$6@f58070b, error [false], resetting" in the logs.

Note that every other query works flawlessly. This is only reproduceable when using the arguments pre_offset and/or post_offset in a DateHistogram with a negative value and with more than 1 node in the cluster.

Facts so far:

  • Gist with the output of _nodes?jvm=true&pretty.
  • Gist with sample query
  • Same query, but without negative values in pre_offset and post_offset works (it doesn't return the results we expect, of course, but no errors/warnings are shown in the logs)
  • This previous issue doesn't seem to be related, there are no other nodes in the network and there are no "custom" java clients around either.
  • The sample query works when there is only one node in the cluster and we start to get these error messages when adding more nodes (2 will suffice to reproduce the issue).

Any ideas? In the meantime, we solved the issue by using pre_zone and post_zone instead of pre_offset and post_offset. Negative values are ok, and everything is running smoothly with those (we tried index, search, and snapshot operations). No error messages in the logs.

Thanks in advance,

@clintongormley clintongormley assigned jpountz and colings86 and unassigned jpountz Aug 18, 2014
@colings86 colings86 changed the title DateHistogram with negative 'pre_offset' or 'post_offset' value ends with "Message not fully read (response) for" Aggregations: DateHistogram with negative 'pre_offset' or 'post_offset' value ends with "Message not fully read (response) for" Aug 18, 2014
@marcelog
Copy link
Author

Wow that was really fast! Thanks :)

@colings86 colings86 removed the v1.3.3 label Aug 19, 2014
colings86 added a commit that referenced this issue Aug 21, 2014
Changes the serialisation of pre and post offset to use Long instead of VLong so that negative values are supported.  This actually only showed up in the case where minDocCount=0 as the rounding is only serialised in this case.

Closes #7312
spinscale pushed a commit that referenced this issue Sep 3, 2014
On Debian amd64, oracle jdk .deb packages made using make-jpkg (from
java-package) default to /usr/lib/jvm/jdk-7-oracle-x64.

Closes #7312
spinscale pushed a commit that referenced this issue Sep 3, 2014
On Debian amd64, oracle jdk .deb packages made using make-jpkg (from
java-package) default to /usr/lib/jvm/jdk-7-oracle-x64.

Closes #7312
colings86 added a commit that referenced this issue Sep 8, 2014
Changes the serialisation of pre and post offset to use Long instead of VLong so that negative values are supported.  This actually only showed up in the case where minDocCount=0 as the rounding is only serialised in this case.

Closes #7312
areek pushed a commit that referenced this issue Sep 8, 2014
On Debian amd64, oracle jdk .deb packages made using make-jpkg (from
java-package) default to /usr/lib/jvm/jdk-7-oracle-x64.

Closes #7312
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants