-
Notifications
You must be signed in to change notification settings - Fork 698
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.
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
Support DocumentDB by replacing certain MongoDB operations #4886
base: master
Are you sure you want to change the base?
Conversation
Add return int values for paging Signed-off-by: DongYoung Kim <[email protected]>
$facet to $group and $project Change the result type. return type has changed Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Change the result type. return type has changed Extract common pagination feature Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $count Change the result type. return type has changed Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Change the result type. return type has changed Extract common pagination feature Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Change the result type. return type has changed Extract common pagination feature Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Change the result type. return type has changed Extract common pagination feature Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group and $project Signed-off-by: DongYoung Kim <[email protected]>
Replace $facet to $group, $project and $unwind Signed-off-by: DongYoung Kim <[email protected]>
Add a condition to check for null pointer error if the res value is no present Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Can you check a |
Signed-off-by: DongYoung Kim <[email protected]>
yes, i will check |
Signed-off-by: DongYoung Kim <[email protected]>
i find a error when get ListExperimentRun, i will fix it |
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Signed-off-by: DongYoung Kim <[email protected]>
Hi @kwx4957 can you perform a benchmarking test to check how these changes have affected the API latency? You can use a mongo compass to run the mongo pipelines and check the latencie==y. |
Otherwise the changes look good to me! 🚀 |
@Saranya-jena Thank you for informing me about the tool I wasn't aware of. yes I will run a benchmarking test. |
What is changed
ChaosCenter
Auth Server
CreatePaginationStage
and add int return values$facet
to$group
and$project
total
and retrieve data by creating an array using$push
with$$ROOT
, then slice the data with$slice
GraphQL Server
CreatePaginationStage
5 Types
$facet
to$count
$facet
to$group
$switch
to count data based on specific cases ingroupbyTotal
$facet
to$group
and$project
total
and retrieve data by creating an array using$push
with$$ROOT
, then slice the data with$slice
$facet
to$group
,$project
, and$unwind
$bucket
to$switch
avg_resiliency_score
Test Code
chaos_experiment_run
AWS documentDB e2e test result
I ran the e2e test on aws documnetdb instead of mongodb, and got the following results
The Google Drive link below is videos of the e2e test.
https://drive.google.com/drive/folders/1_Rs8TPVlOg7KKDCRqzlgtePDO9Kp4I0O?usp=drive_link
Run simple Experiment
i ran simple chaosexperiment using k6. this is the result.
Related proposal: #4812
Related issue: #4459