Skip to content

Improve logging for Optional<Duration> fields. #22

Description

@alec-heif

There seems to be a bug in the logging for fields of type Optional<Duration> in at least the fields for QueryStatistics.

When examining the logs for a query I see that the Duration fields cpuTime, wallTime, and queuedTime are all logged correctly as

      "cpuTime": {
        "seconds": 123,
        "units": [
          "SECONDS",
          "NANOS"
        ],
        "zero": false,
        "negative": false,
        "nano": 123456789
      },
      "wallTime": {
        "seconds": 123,
        "units": [
          "SECONDS",
          "NANOS"
        ],
        "zero": false,
        "negative": false,
        "nano": 123456789
      },
      "queuedTime": {
        "seconds": 123,
        "units": [
          "SECONDS",
          "NANOS"
        ],
        "zero": false,
        "negative": false,
        "nano": 123456789
      }

but the Optional<Duration> fields seem to have the contained value dropped:

      "scheduledTime": {
        "empty": false,
        "present": true
      },
      "analysisTime": {
        "empty": false,
        "present": true
      },
      "planningTime": {
        "empty": false,
        "present": true
      },
      "executionTime": {
        "empty": false,
        "present": true
      }

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions