Skip to content

feat: mixed isolation level anomaly detection#50

Open
dinream wants to merge 16 commits intoTencent:coo-consistency-checkfrom
dinream:coo-consistency-check
Open

feat: mixed isolation level anomaly detection#50
dinream wants to merge 16 commits intoTencent:coo-consistency-checkfrom
dinream:coo-consistency-check

Conversation

@dinream
Copy link

@dinream dinream commented Jul 12, 2024

link: #44

@tencent-adm
Copy link
Member

tencent-adm commented Jul 12, 2024

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hldgs
Copy link

hldgs commented Jul 23, 2024

Please describe which issue you resolved in english

@dinream
Copy link
Author

dinream commented Jul 23, 2024

Please describe which issue you resolved in english
The Issue #44: Mixed isolation level anomaly detection

@hldgs
Copy link

hldgs commented Jul 24, 2024

Please describe which issue you resolved in english
The Issue #44: Mixed isolation level anomaly detection

sorry, i mean add this in your pr description

@dinream dinream changed the title Coo consistency check Issue #44: Mixed isolation level anomaly detection Jul 25, 2024
@dinream
Copy link
Author

dinream commented Jul 25, 2024

Please describe which issue you resolved in english
The Issue #44: Mixed isolation level anomaly detection

sorry, i mean add this in your pr description

Yeah, I understand. I have changed it

@hldgs
Copy link

hldgs commented Jul 25, 2024

you can use "feat: mixed isolation level anomaly detection" as your pr title, and add "link: #50 "as your pr description
image

@dinream dinream changed the title Issue #44: Mixed isolation level anomaly detection feat: mixed isolation level anomaly detection Jul 25, 2024
@dinream
Copy link
Author

dinream commented Jul 25, 2024

you can use "feat: mixed isolation level anomaly detection" as your pr title, and add "link: #50 "as your pr description image

OK


Returns:
None
"""
Copy link

Choose a reason for hiding this comment

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

dont delete our docstring

Copy link
Author

Choose a reason for hiding this comment

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

i have restore all docstring

visit[e.out] = 0


# # for loop graphs, print the loop
Copy link

Choose a reason for hiding this comment

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

if code is unnecessary, just delete it. dont add these comments

Copy link
Author

Choose a reason for hiding this comment

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

ok

f.write("\n\n")


run_result_folder = "pg/serializable"
Copy link

Choose a reason for hiding this comment

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

this part need more review, you can describe it in detail, and we can discuss it further

Copy link
Author

Choose a reason for hiding this comment

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

At present, my main idea is to optimize the code edge strategy according to different isolation levels. Can you describe in detail which aspects I will explain in detail?

Copy link
Author

Choose a reason for hiding this comment

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

Or if you need, I can explain it to you in a more convenient way, like setting up a time for an online meeting.

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.

3 participants