Skip to content
Snippets Groups Projects

add schema parse error in record trace field

Merged Neelesh Thakur requested to merge schema-errors into master

Address issue: 27

Add schema parse errors in record trace field on indexed records. Users can later use Search service to query records and check errors themselves.

As an example, in case of schema parsing failure, a generic message get's indexed on the record instead of actual error message:

 "results": [
    {
        "index": {
            "trace": [
                "schema not found"
            ],
            "statusCode": 404,
            "lastUpdateTime": "2021-06-24T14:23:50.596Z"
        },
        "id": "opendes:indexer-test:01ae8501-1534-4c6b-93e7-8ea2ed26d752.xml"
    }
]

This MR addresses this and indexes specific error messages (highlighted in bold), enabling ingestor to take corrective actions:

 "results": [
    {
        "index": {
            "trace": [
                "Errors occurred during parsing the schema, kind: opendes:indexer-test:sample-schema-2:2.0.0 | errors: kind cannot be null or empty for path 'WB_NAME'"
            ],
            "statusCode": 400,
            "lastUpdateTime": "2021-06-24T14:23:50.596Z"
        },
        "id": "opendes:indexer-test:01ae8501-1534-4c6b-93e7-8ea2ed26d752.xml"
    }
]

Instruction to search trace field can be found here.

Edited by Neelesh Thakur

Merge request reports

Merge request pipeline #52167 failed

Pipeline: Indexer

#52168

    Merge request pipeline failed for 679beb84

    Failed to deploy to A‎WS‎
    Deployed to I‎BM‎ 3 years ago

    Merged by Neelesh ThakurNeelesh Thakur 3 years ago (Jul 14, 2021 3:38pm UTC)

    Merge details

    Pipeline #52223 failed

    Pipeline failed for a81895c5 on master

    Skipped deployment to G‎CP‎
    Deployed to I‎BM‎ 3 years ago
    Failed to deploy to A‎WS‎

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Neelesh Thakur changed the description

    changed the description

  • Greg approved this merge request

    approved this merge request

  • Spencer Sutton approved this merge request

    approved this merge request

  • Neelesh Thakur added 1 commit

    added 1 commit

    Compare with previous version

  • Neelesh Thakur resolved all threads

    resolved all threads

  • Ankit Sharma [Microsoft] approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading