Skip to content

Add Feature -Redis Exporter and Exporter Interface Variable Update For Zeebe#272

Open
aamirmousavi wants to merge 2 commits intonitram509:mainfrom
aamirmousavi:main
Open

Add Feature -Redis Exporter and Exporter Interface Variable Update For Zeebe#272
aamirmousavi wants to merge 2 commits intonitram509:mainfrom
aamirmousavi:main

Conversation

@aamirmousavi
Copy link
Contributor

Motivation/Abstract

Our primary motivation is to utilize Redis as a supported exporter for Zeebe to meet specific project requirements and to improve the testability and monitoring of BPMN flows by allowing direct variable manipulation. This PR introduces a functional Redis exporter and extends the Exporter interface with an updateVariable method (implemented for both Hazelcast and Redis) to address these needs.

Description/Comments

This pull request delivers two key enhancements:

  • Redis Exporter Implementation:
    We've developed a Redis exporter because our operational setup requires its use over Hazelcast. The implementation ensures that Zeebe records can be effectively streamed to and stored in Redis, aligning with Zeebe's official support for Redis as an alternative exporter.
  • updateVariable in Exporter Interface:
    To facilitate more robust testing and monitoring of our BPMN processes, especially when needing to simulate specific states or debug issues related to process variables, we've added an updateVariable method to the Exporter interface. This new method has been implemented for both the existing Hazelcast exporter and the newly introduced Redis exporter, providing a consistent way to modify variables regardless of the chosen exporter.
    These changes aim to offer more flexibility in exporter choices and enhance the developer experience around testing and process instance management.

#271

Tests

  • did you update or create tests for your code changes?
  • not relevant

Code examples

  • did you update or add example code snippets, which relate to your code changes
  • not relevant

Documentation

  • did you update or create documentation, which relates to your code changes
  • not relevant

@codecov
Copy link

codecov bot commented May 28, 2025

Codecov Report

❌ Patch coverage is 4.71204% with 182 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.07%. Comparing base (e5b2c83) to head (ef1fe30).
⚠️ Report is 29 commits behind head on main.

Files with missing lines Patch % Lines
pkg/bpmn_engine/exporter/zeebe/redis_exporter.go 0.00% 158 Missing ⚠️
pkg/bpmn_engine/exporter/zeebe/zeebe_exporter.go 0.00% 21 Missing ⚠️
pkg/bpmn_engine/exporter.go 72.72% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #272      +/-   ##
==========================================
- Coverage   63.44%   58.07%   -5.38%     
==========================================
  Files          28       29       +1     
  Lines        1896     2087     +191     
==========================================
+ Hits         1203     1212       +9     
- Misses        647      828     +181     
- Partials       46       47       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant