-
Notifications
You must be signed in to change notification settings - Fork 308
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: introduce pipeline
crate
#4109
feat: introduce pipeline
crate
#4109
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4109 +/- ##
==========================================
- Coverage 85.47% 85.16% -0.31%
==========================================
Files 993 993
Lines 173751 173751
==========================================
- Hits 148505 147968 -537
- Misses 25246 25783 +537 |
pipeline
cratepipeline
crate
This is of course a new feature rather than "chore" :D |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The original pipeline repo has a detailed pr-review history. Just merge this mv operation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a huge PR. But let's move forward.
}; | ||
use crate::etl::value::{Map, Value}; | ||
|
||
pub(crate) const PROCESSOR_CMCD: &str = "cmcd"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does "cmcd" stand for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found "Common Media Client Data" below now :D
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
This is a pre-pr for #4014 to introduce https://github.com/GreptimeTeam/pipeline into
src/pipeline/src/etl
. The code is not referred anywhere now.Checklist