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.

Sign up for GitHub

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

Jump to bottom

fix: Added span and test for TRANSACTION_RUN_QUERY #1496

Closed
wants to merge 4 commits into from
Closed

fix: Added span and test for TRANSACTION_RUN_QUERY #1496

wants to merge 4 commits into from

Commits on Jun 25, 2024

  1. fix: Fixed the TraceUtil.startSpan method to use SpanContext for li… 

    …nking with the parent instead of `Context`.

- This fixes the hierarchy of Spans appearing in a transaction under a Run method.
- Tested using existing transaction test
    @jimit-j-shah
    jimit-j-shah committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    76df271 View commit details
    Browse the repository at this point in the history

  2. Fixed commit reordering and typos

    @jimit-j-shah
    jimit-j-shah committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    21e59d9 View commit details
    Browse the repository at this point in the history

  3. fix: Added span and test for TRANSACTION_RUN_QUERY 

    - unit tested
    @jimit-j-shah
    jimit-j-shah committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    62b7fbe View commit details
    Browse the repository at this point in the history

  4. feat: Added span for Transactional RunQuery 

    - tested
    @jimit-j-shah
    jimit-j-shah committed Jun 25, 2024
    Configuration menu
    Copy the full SHA
    7fdd0ec View commit details
    Browse the repository at this point in the history