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

Event Clean-Up #6716

Merged
merged 2 commits into from
Jan 25, 2023
Merged

Event Clean-Up #6716

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
69de15e
CT-1857: Event cleanup
peterallenwebb Jan 24, 2023
525e3d0
Add changelog entry.
peterallenwebb Jan 24, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .changes/unreleased/Under the Hood-20230124-153553.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
kind: Under the Hood
body: Revised and simplified various structured logging events
time: 2023-01-24T15:35:53.065356-05:00
custom:
Author: peterallenwebb
Issue: 6664 6665 6666
175 changes: 29 additions & 146 deletions core/dbt/events/proto_types.py
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

137 changes: 24 additions & 113 deletions core/dbt/events/types.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -863,98 +863,7 @@ message ParsedFileLoadFailedMsg {
ParsedFileLoadFailed data = 2;
}

// Skipping I030


// I031
message StaticParserCausedJinjaRendering {
string path = 1;
}

message StaticParserCausedJinjaRenderingMsg {
EventInfo info = 1;
StaticParserCausedJinjaRendering data = 2;
}

// I032
message UsingExperimentalParser {
string path = 1;
}

message UsingExperimentalParserMsg {
EventInfo info = 1;
UsingExperimentalParser data = 2;
}

// I033
message SampleFullJinjaRendering {
string path = 1;
}

message SampleFullJinjaRenderingMsg {
EventInfo info = 1;
SampleFullJinjaRendering data = 2;
}

// I034
message StaticParserFallbackJinjaRendering {
string path = 1;
}

message StaticParserFallbackJinjaRenderingMsg {
EventInfo info = 1;
StaticParserFallbackJinjaRendering data = 2;
}

// I035
message StaticParsingMacroOverrideDetected {
string path = 1;
}

message StaticParsingMacroOverrideDetectedMsg {
EventInfo info = 1;
StaticParsingMacroOverrideDetected data = 2;
}

// I036
message StaticParserSuccess {
string path = 1;
}

message StaticParserSuccessMsg {
EventInfo info = 1;
StaticParserSuccess data = 2;
}

// I037
message StaticParserFailure {
string path = 1;
}

message StaticParserFailureMsg {
EventInfo info = 1;
StaticParserFailure data = 2;
}

// I038
message ExperimentalParserSuccess {
string path = 1;
}

message ExperimentalParserSuccessMsg {
EventInfo info = 1;
ExperimentalParserSuccess data = 2;
}

// I039
message ExperimentalParserFailure {
string path = 1;
}

message ExperimentalParserFailureMsg {
EventInfo info = 1;
ExperimentalParserFailure data = 2;
}
// Skipping I030 - I039

// I040
message PartialParsingEnabled {
Expand Down Expand Up @@ -1124,6 +1033,28 @@ message JinjaLogWarningMsg {
JinjaLogWarning data = 2;
}

// I062
message JinjaLogInfo {
NodeInfo node_info = 1;
string msg = 2;
}

message JinjaLogInfoMsg {
EventInfo info = 1;
JinjaLogInfo data = 2;
}

// I063
message JinjaLogDebug {
NodeInfo node_info = 1;
string msg = 2;
}

message JinjaLogDebugMsg {
EventInfo info = 1;
JinjaLogDebug data = 2;
}

// M - Deps generation

// M001
Expand Down Expand Up @@ -1230,27 +1161,7 @@ message SelectorReportInvalidSelectorMsg {
SelectorReportInvalidSelector data = 2;
}

// M011
message JinjaLogInfo {
NodeInfo node_info = 1;
string msg = 2;
}

message JinjaLogInfoMsg {
EventInfo info = 1;
JinjaLogInfo data = 2;
}

// M012
message JinjaLogDebug {
NodeInfo node_info = 1;
string msg = 2;
}

message JinjaLogDebugMsg {
EventInfo info = 1;
JinjaLogDebug data = 2;
}
// Skipped M011 and M012

// M013
message DepsNoPackagesFound {
Expand Down
Loading