Skip to content

Conversation

@RotemFB
Copy link
Contributor

@RotemFB RotemFB commented Jul 5, 2023

added a timer option that counts the time of Authentication, Resolving engine name, Running query and Parsing query output. the info will be written on the log if an env variable named FIREBOLT_SDK_PERFORMANCE_DEBUG will be equal to 1

@RotemFB RotemFB requested a review from stepansergeevitch July 5, 2023 08:45
@RotemFB RotemFB self-assigned this Jul 5, 2023
@RotemFB RotemFB requested a review from stepansergeevitch July 5, 2023 14:14
Copy link
Contributor

@ptiurin ptiurin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@RotemFB RotemFB force-pushed the performance_debug branch from 4c1be48 to c578fe1 Compare July 13, 2023 08:34
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

88.2% 88.2% Coverage
0.0% 0.0% Duplication

@RotemFB RotemFB merged commit 49381f8 into main Jul 13, 2023
@RotemFB RotemFB deleted the performance_debug branch July 13, 2023 08:40
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.

4 participants