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

Updates tantivy for bugfix in json value serializaton. #1415

Merged
merged 1 commit into from
May 11, 2022
Merged

Conversation

fulmicoton
Copy link
Collaborator

The target PR in tantivy is quickwit-oss/tantivy#1367

Closes #1411

How was this PR tested?

A unit test was added on tantivy side and the ticket example has been tested and yielded the following result.

{
  "num_hits": 3,
  "hits": [
    {
      "attributes": {
        "syslog": {
          "facility": "daemon",
          "procid": 7816,
          "version": 2
        }
      },
      "body": "<26>2 2022-05-10T17:59:56.706Z for.us benefritz 7816 ID715 - A bug was encountered but not in Vector, which doesn't have bugs",
      "name": "ID715",
      "severity": "ERROR",
      "timestamp": 1652205596
    },
    {
      "attributes": {
        "syslog": {
          "facility": "daemon",
          "procid": 7816,
          "version": 2
        }
      },
      "body": "<26>2 2022-05-10T17:59:56.706Z for.us benefritz 7816 ID715 - A bug was encountered but not in Vector, which doesn't have bugs",
      "name": "ID715",
      "severity": "ERROR",
      "timestamp": 1652205596
    },
    {
      "attributes": {
        "syslog": {
          "facility": "daemon",
          "procid": 7816,
          "version": 2
        }
      },
      "body": "<26>2 2022-05-10T17:59:56.706Z for.us benefritz 7816 ID715 - A bug was encountered but not in Vector, which doesn't have bugs",
      "name": "ID715",
      "severity": "ERROR",
      "timestamp": 1652205596
    }
  ],
  "elapsed_time_micros": 25257,
  "errors": []
}

@fulmicoton fulmicoton merged commit b9ffb33 into main May 11, 2022
@fulmicoton fulmicoton deleted the issue/1411 branch May 11, 2022 04:36
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.

Error when fetching docs with a json field defined in the doc mapper
1 participant