Skip to content

Make QueryEngine code more readable by abstracting out debug and panic statements #100

Closed
#342
Closed
Make QueryEngine code more readable by abstracting out debug and panic statements#100
#342
Assignees
milindsrivastava1997
Labels
asap-query-engine

Description

@milindsrivastava1997
milindsrivastava1997
opened on Mar 4, 2026

Bunch of debug statements for latency tracking can be replaced by tracing::debug_span!

Bunch of panic statements should probably be replaced by returning an Err instead

Activity

  1. milindsrivastava1997 commented on Mar 4, 2026

    @milindsrivastava1997
    milindsrivastava1997
    on Mar 4, 2026
    ContributorAuthor

    Originally by @linear[bot] on 2026-01-01:

    PRO-350

  2. milindsrivastava1997
    added
    asap-query-engine
    on Mar 4, 2026
  3. milindsrivastava1997
    self-assigned this
    on May 19, 2026
  4. milindsrivastava1997
    linked a pull request that will close this issuerefactor(query-engine): replace panics with Result/None #342on May 19, 2026
  5. milindsrivastava1997
    closed this as completedin #342on May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

asap-query-engine

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions