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.

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

feat: add properties to #[trace] #190

Merged
merged 7 commits into from
Jan 25, 2024
Merged

feat: add properties to #[trace] #190

merged 7 commits into from
Jan 25, 2024

Conversation

andylokandy
Copy link
Collaborator

@andylokandy andylokandy commented Jan 24, 2024

Signed-off-by: Andy Lok [email protected]

Closes #183

Signed-off-by: Andy Lok <[email protected]>
Signed-off-by: Andy Lok <[email protected]>
@coveralls
Copy link

coveralls commented Jan 24, 2024

Pull Request Test Coverage Report for Build 7644903030

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.8%) to 79.068%

Totals Coverage Status
Change from base Build 7640485360: -1.8%
Covered Lines: 1696
Relevant Lines: 2145

💛 - Coveralls

@andylokandy andylokandy requested a review from zhongzc January 24, 2024 18:45
Signed-off-by: Andy Lok <[email protected]>
Signed-off-by: Andy Lok <[email protected]>
Signed-off-by: Andy Lok <[email protected]>
Signed-off-by: Andy Lok <[email protected]>
Comment on lines +304 to +306
if args.enter_on_poll {
abort_call_site!("`enter_on_poll` can not be used with `properties`")
}
Copy link
Collaborator

@zhongzc zhongzc Jan 25, 2024

Choose a reason for hiding this comment

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

see it but regret.

Comment on lines +62 to +63
- [Apache OpenDAL](https://github.com/apache/opendal): A data access layer for various storage
- [Databend](https://github.com/datafuselabs/databend): Cost-Effective alternative to Snowflake
Copy link
Collaborator

Choose a reason for hiding this comment

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

🎉

@andylokandy andylokandy merged commit 894d548 into tikv:master Jan 25, 2024
8 checks passed
@dotansimha
Copy link
Contributor

Thank you @andylokandy !

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.

Feature request: Span property on #[trace] macro
4 participants