Skip to content

Conversation

de-sh
Copy link
Contributor

@de-sh de-sh commented Dec 1, 2024

Fixes #XXXX.

Using tracing will allow us to capture span related info in addition to the event based context that we already capture

Description


This PR has:

  • been tested to ensure log ingestion and log query works.
  • added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added documentation for new or modified features or behaviors.

@de-sh de-sh mentioned this pull request Dec 1, 2024
3 tasks
nitisht pushed a commit that referenced this pull request Dec 2, 2024
Fixes ci issues pointed out in #1013. This is pertaining 
to lints which have been introduced in the latest version 
of clippy.
@de-sh de-sh marked this pull request as ready for review December 5, 2024 11:00
@nitisht nitisht merged commit acb26b9 into parseablehq:main Dec 9, 2024
8 checks passed
@de-sh de-sh deleted the tracing branch December 9, 2024 14:54
de-sh pushed a commit to de-sh/parseable that referenced this pull request Dec 20, 2024
Using tracing will allow us to capture span related info in addition to the event based context that we already capture

---------

Signed-off-by: Devdutt Shenoi <[email protected]>
nitisht pushed a commit that referenced this pull request Dec 20, 2024
…nd sync steps (#1039)

* refactor: `log` ~> `tracing` (#1013)

Using tracing will allow us to capture span related info in addition to the event based context that we already capture

---------

Signed-off-by: Devdutt Shenoi <[email protected]>

* note span creation and deletion

* trace: log steps of ingest and sync

---------

Signed-off-by: Devdutt Shenoi <[email protected]>
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.

3 participants