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

NMS-13115: add FlowSummaryData for improved performance #13

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

swachter
Copy link
Contributor

@swachter swachter commented Jan 28, 2021

Flow summaries are determined by creating the "union" over multiple PCollections that contain the flow summaries for certain aggregations. The union operation requires that flow summary information is encoded / decoded. At the moment a JacksonJsonCoder is used that is not efficient because it is based on reflection and uses JSON as the representation for transmission.

Considering the expected amount of generated flow summaries an efficient custom coder for flow summary information should be used. The JSON representation of flow summaries is created at the output stages of the pipeline, i.e. when they are written to Elastic and Kafka.

@swachter swachter requested a review from fooker January 28, 2021 17:19
@swachter swachter changed the title no-jira: add FlowSummaryData for improved performance NMS-13115: add FlowSummaryData for improved performance Feb 2, 2021
Copy link
Member

@fooker fooker left a comment

Choose a reason for hiding this comment

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

Just minor improvements.

@swachter swachter requested a review from fooker February 2, 2021 13:27
@swachter swachter merged commit fa589fe into master Feb 2, 2021
@swachter swachter deleted the swachter/delay-populate branch February 2, 2021 15:03
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.

2 participants