You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
New parsers should NOT annotate records, as they are annotated by joins in BQ.
The K8S annotation-service should be shut down, and null-annotator should be used for 2.0 parsing tasks.
This would also make the annotator metrics easier to interpret, as they would be legacy pipeline only.
The text was updated successfully, but these errors were encountered:
@gfr10598 can you clarify? We discussed part of this last week. Is this more about cleaning up the annotation metrics? The 2.0 parsers (ndt7 & annotation) do not use the annotation service today. Please update the issue title/description to clarify.
Working on the local writer for ETL, I discovered that ndt7 (and probably annotation) attempt to contact the annotation-service unnecessarily. The NewSinkParser for ndt7 and annotation services receives a real api.Annotator.
New parsers should NOT annotate records, as they are annotated by joins in BQ.
The K8S annotation-service should be shut down, and null-annotator should be used for 2.0 parsing tasks.
This would also make the annotator metrics easier to interpret, as they would be legacy pipeline only.
The text was updated successfully, but these errors were encountered: