Skip to content

Conversation

@gukj-spel
Copy link

@gukj-spel gukj-spel commented Jul 15, 2024

Modify the .gitgnore and correct a function comment:

  • change .gitignore to ignore build and .vscode folder
  • correct sqltest_v2.cc MultiThreadExecution method comments

@hldgs
Copy link

hldgs commented Jul 23, 2024

can you edit your pr title to "fix: correct .gitignore and some comments"?

@gukj-spel
Copy link
Author

can you edit your pr title to "fix: correct .gitignore and some comments"?

sure

@gukj-spel gukj-spel changed the title fix some little bug fix: correct .gitignore and some comments Jul 23, 2024
Copy link

@hldgs hldgs left a comment

Choose a reason for hiding this comment

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

if any questions, reply me directly

@@ -1,5 +1,5 @@
build/*
.vscode/*
*build*/
Copy link

Choose a reason for hiding this comment

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

why "*build*/" ? I think "build/" is sufficient.

* @param db_type The type of the database (e.g., "oracle", "mysql").
* @param test_process_file The file to log test process.
* @return True if the query execution is successful, false otherwise.
* Executes multiple SQL transactions in a multi-threaded environment.
Copy link

Choose a reason for hiding this comment

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

"Executes multiple SQL transactions", the rest is unnecessary in this line

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.

2 participants