Skip to content
This repository has been archived by the owner on Jan 28, 2022. It is now read-only.

Add ability to configure the number of max concurrent reconciles #154

Merged

Conversation

EliiseS
Copy link
Collaborator

@EliiseS EliiseS commented Jan 29, 2020

This update is to address the issue with the operator being unable to handle the load raised here: #153

We re-ran the load test with MaxConcurrentReconciles set 10 instead of the default 1 resulting in noticeable improvements seen below.

Here we can see the where to load ramps up to 25 users(runs)
image

As the work load increases we can see the average wait time climb to 5 seconds vs the previous 10 seconds.
image

The queue depth is also lower than before at around 30 vs around 40 previously.
image

We feel this is a good solution for being able to configure to controller to be able to handle more load.

@stuartleeks
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@stuartleeks
Copy link
Collaborator

Looks good to me.
@Azadehkhojandi - woulc you be able to review this PR?

@Azadehkhojandi
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@Azadehkhojandi Azadehkhojandi left a comment

Choose a reason for hiding this comment

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

Thank you, I will merge the PR if the build passes

@Azadehkhojandi Azadehkhojandi merged commit 984e6ed into Azure:master Feb 17, 2020
EliiseS added a commit to EliiseS/azure-databricks-operator that referenced this pull request Feb 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants