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

Jump to bottom

[v0.7.2] Fix logging #133

Merged
merged 6 commits into from
Mar 16, 2020
Merged

[v0.7.2] Fix logging #133

merged 6 commits into from
Mar 16, 2020

Conversation

ale93p
Copy link
Owner

@ale93p ale93p commented Feb 18, 2020 •
edited

PR Description

This PR updates namb to version 0.7.2:

  • Fixed Logging system: now tuples are sampled based on source count to avoid mismatch along pipeline
  • Refactored BenchmarkApplication component for better code management (Storm and Flink)
  • Refactored Processing Load computation as of CodeRefactoring: compute processing load at init #73

Closing issues

Closes #73

@ale93p
[Storm] Refactored BenchmarkApplication component: splitted TopologyB…
b144e44 
…uilder
@ale93p
[AppBuilder] Compute Processing Load at init (Closes #73)
2b5d100
@ale93p
[Storm] Fixed tuple logging: now tuples are sampled based on source c…
8fe4338 
…ount and not local one

        This assure to have printed always the same tuples, better for latency computation
@ale93p
[Flink] Refactored BenchmarkApplication component: splitted TopologyB…
2057547 
…uilder
@ale93p
[Flink] [Storm] Fixed tuple logging: now tuples are sampled based on …
c87ab4c 
…source count and not local one

            This assure to have printed always the same tuples, better for latency computation
@ale93p ale93p added Type: Bug Something isn't working Type: Enhancement Improvement of existing feature Priority: Critical The issue is a blocker Platform: All The issue is related to all the applications labels Feb 18, 2020
@ale93p ale93p merged commit f480d46 into master Mar 16, 2020
@ale93p ale93p changed the title [WIP] [v0.7.2] Fix logging [v0.7.2] Fix logging Mar 16, 2020
@ale93p ale93p deleted the fix-logging branch March 16, 2020 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Platform: All The issue is related to all the applications Priority: Critical The issue is a blocker Type: Bug Something isn't working Type: Enhancement Improvement of existing feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CodeRefactoring: compute processing load at init
1 participant
@ale93p