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(processor): add profiling for new processor executor #4932

Closed
wants to merge 6 commits into from

Conversation

zhang2014
Copy link
Member

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

feat(processor): add profiling for executor

Changelog

  • New Feature

Related Issues

Fixes #4238

@vercel
Copy link

vercel bot commented Apr 19, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
databend ⬜️ Ignored (Inspect) Apr 27, 2022 at 2:15AM (UTC)

@zhang2014 zhang2014 changed the title feat(processor): add profiling for executor feat(processor): add profiling for new processor executor Apr 19, 2022
@mergify
Copy link
Contributor

mergify bot commented Apr 19, 2022

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@mergify mergify bot added the pr-feature this PR introduces a new feature to the codebase label Apr 19, 2022
@chinawch007
Copy link

Excuse me.
S3 downloading and deserializing take most of the query executing time so you try to monitor them. Is this conclusion correct?

@mergify
Copy link
Contributor

mergify bot commented Jun 28, 2022

This pull request's description is not fulfill the requirements. @zhang2014 please update it 🙏.

The description should contain the following:

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

## Summary

  Summary about this PR

## Changelog

- New Feature
- Bug Fix
- Improvement
- Performance Improvement
- Build/Testing/CI
- Documentation
- Not for changelog (changelog entry is not required)

## Related Issues

Fixes #issue

@Xuanwo
Copy link
Member

Xuanwo commented Jul 6, 2022

Hi, is this PR still going on? How about closing it until we are ready to start again?

@Xuanwo
Copy link
Member

Xuanwo commented Aug 10, 2022

Let's close this stale PR. Feel free to start new PR when ready~

@Xuanwo Xuanwo closed this Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need-review pr-feature this PR introduces a new feature to the codebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add processor level profiling
4 participants