Skip to content
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.

Sign up for GitHub

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

Jump to bottom

docs:add taos_slow_sql_detail & log principle #29843

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

docs:add taos_slow_sql_detail & log principle #29843

wants to merge 3 commits into from
+152 −56

Conversation

wangmm0220
Copy link
Contributor

Description

Please briefly describe the code changes in this pull request.

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

dapan1121
dapan1121 reviewed Feb 20, 2025
View reviewed changes
docs/en/14-reference/01-components/01-taosd.md Outdated
| process\_id | VARCHAR | | process id |
| db | VARCHAR | TAG | sql db |
| user | VARCHAR | TAG | sql user |
| ip | VARCHAR | TAG | sql ip |
Copy link
Contributor

Choose a reason for hiding this comment

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

sql ip是什么

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

docs/zh/14-reference/01-components/01-taosd.md Outdated
| process\_id | VARCHAR | | 进程 id |
| db | VARCHAR | TAG | 执行 sql 所属数据库 |
| user | VARCHAR | TAG | 执行 sql 语句的用户 |
| ip | VARCHAR | TAG | 记录执行 sql 语句的 ip 地址 |
Copy link
Contributor

Choose a reason for hiding this comment

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

需要再明确一点，到底是谁的IP

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dapan1121 dapan1121 dapan1121 left review comments

@guanshengliang guanshengliang Awaiting requested review from guanshengliang guanshengliang is a code owner

@zitsen zitsen Awaiting requested review from zitsen zitsen is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wangmm0220 @dapan1121