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

Add debug exporter #1240

Merged
merged 1 commit into from Sep 4, 2018

Conversation

2 participants
@menski
Copy link
Member

commented Sep 3, 2018

Adds an exporter implementation for debugging. It will log every
exported record using the logger provided to the exporter. It can be
configured to use a specific log level (default: debug) and if records
should be pretty printed for better readability.

The debugger can be enabled for every embedded broker/clustering rule
during testing by setting the ENABLE_DEBUG_EXPORTER to true.

The default distro configuration contains a exporter example for the
debug exporter so users can easily enable it.

The debug exporter does not update the exported records position by
intention so every record is always printed even on restart of the
broker.

closes #1239

@menski menski requested a review from Zelldon Sep 3, 2018

@ghost ghost assigned menski Sep 3, 2018

@ghost ghost added the needs review label Sep 3, 2018

feat(broker): add debug exporter
Adds an exporter implementation for debugging. It will log every
exported record using the logger provided to the exporter. It can be
configured to use a specific log level (default: debug) and if records
should be pretty printed for better readability.

The debugger can be enabled for every embedded broker/clustering rule
during testing by setting the ENABLE_DEBUG_EXPORTER to true.

The default distro configuration contains a exporter example for the
debug exporter so users can easily enable it.

The debug exporter does not update the exported records position by
intention so every record is always printed even on restart of the
broker.
@Zelldon

Zelldon approved these changes Sep 4, 2018

@Zelldon

This comment has been minimized.

Copy link
Member

commented Sep 4, 2018

bors r+

bors bot added a commit that referenced this pull request Sep 4, 2018

Merge #1240
1240: Add debug exporter r=Zelldon a=menski

Adds an exporter implementation for debugging. It will log every
exported record using the logger provided to the exporter. It can be
configured to use a specific log level (default: debug) and if records
should be pretty printed for better readability.

The debugger can be enabled for every embedded broker/clustering rule
during testing by setting the ENABLE_DEBUG_EXPORTER to true.

The default distro configuration contains a exporter example for the
debug exporter so users can easily enable it.

The debug exporter does not update the exported records position by
intention so every record is always printed even on restart of the
broker.

closes #1239


Co-authored-by: Sebastian Menski <sebastian@menski.org>
@bors

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2018

@Zelldon

This comment has been minimized.

Copy link
Member

commented Sep 4, 2018

bors retry

bors bot added a commit that referenced this pull request Sep 4, 2018

Merge #1240
1240: Add debug exporter r=Zelldon a=menski

Adds an exporter implementation for debugging. It will log every
exported record using the logger provided to the exporter. It can be
configured to use a specific log level (default: debug) and if records
should be pretty printed for better readability.

The debugger can be enabled for every embedded broker/clustering rule
during testing by setting the ENABLE_DEBUG_EXPORTER to true.

The default distro configuration contains a exporter example for the
debug exporter so users can easily enable it.

The debug exporter does not update the exported records position by
intention so every record is always printed even on restart of the
broker.

closes #1239


Co-authored-by: Sebastian Menski <sebastian@menski.org>
@bors

This comment has been minimized.

Copy link
Contributor

commented Sep 4, 2018

@bors bors bot merged commit 5e8ca25 into develop Sep 4, 2018

3 checks passed

bors Build succeeded
Details
continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/jenkins/pr-head This commit looks good
Details

@bors bors bot deleted the 1239-debug-exporter branch Sep 4, 2018

@ghost ghost removed the needs review label Sep 4, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.