cxing opened a new pull request, #9636:
URL: https://github.com/apache/pinot/pull/9636
Instructions:
1. The PR has to be tagged with at least one of the following labels (*):
`performance`
2. Purpose for this pull request:
We found a performance issue from master pinot code
When we use zk as user and password authorization we found this function
spend about 80ms for verifying
```isMatch = BCrypt.checkpw(pasword, encrypedPassword)```
So this solution is to cache this function result to enhance query speed
3. Here is testing result in our lab env:
Original code need 85ms at least to query
Fixed code need 1 - 10ms at least to query
4. Related issues: https://github.com/apache/pinot/issues/9632
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
