Skip to content

Conversation

@jandadav
Copy link
Contributor

@jandadav jandadav commented Aug 27, 2020

Description

Improve performance of query endpoint and ZAAS client

The ZAAS client had query performance measured on SDK sample including ZAAS client around 700ms. With heavy load, we measured a performance degradation up to 10x (7s per request)

After the fix, with heavy load, the requests take around 300ms and do not scale with load

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

@jandadav jandadav changed the title Rip/dj/auth performance evaluation Improve performance of query endpoint and ZAAS client Aug 27, 2020
@jandadav jandadav requested a review from achmelo August 27, 2020 11:41
this fixes a cache miss

Signed-off-by: jandadav <janda.david@gmail.com>
Signed-off-by: jandadav <janda.david@gmail.com>
@jandadav jandadav force-pushed the rip/dj/auth-performance-evaluation branch from 8315ed0 to 0253517 Compare August 27, 2020 11:52
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

94.7% 94.7% Coverage
0.0% 0.0% Duplication

warning The version of Java (1.8.0_242) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@jandadav jandadav requested a review from taban03 August 27, 2020 13:50
@jandadav jandadav merged commit 8e96c59 into master Aug 27, 2020
@delete-merged-branch delete-merged-branch bot deleted the rip/dj/auth-performance-evaluation branch August 27, 2020 13:58
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