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

2.3.0 alpha 2 search only returning 150 items #3813

Closed
kpointer opened this issue May 11, 2017 · 1 comment
Closed

2.3.0 alpha 2 search only returning 150 items #3813

kpointer opened this issue May 11, 2017 · 1 comment
Assignees
Labels
Milestone

Comments

@kpointer
Copy link

@kpointer kpointer commented May 11, 2017

Expected Behavior

Search should return all items in the range.

Current Behavior

Searching only returning 150 items. The Histogram graphs have the right number of logs.

Search result

Found 150 messages in 4 ms, searched in 2 indices.

Possible Solution

Search is broken in 2.3.0 alpha 2

Steps to Reproduce (for bugs)

  1. Do a search

Context

Testing alpha 2

Your Environment

  • Graylog Version: Graylog 2.3.0-alpha.2+f24e575
  • Elasticsearch Version: 5.4.0
  • MongoDB Version: 3.4.4
  • Operating System: Ubuntu 16.04.02 LTS
  • Browser version: Firefox 54.0b6 and Safari 10.1.1
@kpointer
Copy link
Author

@kpointer kpointer commented May 11, 2017

Did a little more digging. 2.3.0 alpha 2 is doing the following query:

{
  "size": 150,
  "query": {
    "bool": {
      "must": {
        "match_all": {}
      },
      "filter": {
        "bool": {
          "must": {
            "range": {
              "timestamp": {
                "from": "2017-05-10 15:33:08.094",
                "to": "2017-05-11 15:33:08.094",
                "include_lower": true,
                "include_upper": true
              }
            }
          }
        }
      }
    }
  },
  "sort": [
    {
      "timestamp": {
        "order": "desc"
      }
    }
  ]
}

2.2.3 is doing the following:

{
  "from": 0,
  "size": 150,
  "query": {
    "bool": {
      "must": {
        "match_all": {}
      },
      "filter": {
        "bool": {
          "must": {
            "range": {
              "timestamp": {
                "from": "2017-05-10 15:32:02.530",
                "to": "2017-05-11 15:32:02.530",
                "include_lower": true,
                "include_upper": true
              }
            }
          }
        }
      }
    }
  },
  "sort": [
    {
      "timestamp": {
        "order": "desc"
      }
    }
  ]
}

The 2.3.0 doesn't have the "from": 0, included.

joschi pushed a commit that referenced this issue May 12, 2017
Jochen Schalanda
Fixes #3813
@joschi joschi self-assigned this May 12, 2017
@joschi joschi added the bug label May 12, 2017
@joschi joschi added this to the 2.3.0 milestone May 12, 2017
@joschi joschi added the in progress label May 12, 2017
@jalogisch jalogisch added the triaged label May 15, 2017
@ghost ghost removed the in progress label May 15, 2017
dennisoelkers added a commit that referenced this issue May 15, 2017
* Fix pagination and offset/total hits in Searches

Fixes #3813

* Add test for Search#searches(...)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.