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

DEV-270 - Improve configuration of transform and partitioner #85

Conversation

josephcummings
Copy link
Contributor

@josephcummings josephcummings commented Feb 9, 2024

Changed: Update and improved configuration of a transform and custom partitioner, which particularly benefits helm deployment when either are configured.
Added: Parsing of file paths for both transformJs and partitionerJs to prevent mismatch between config and volume mount

Documentation PR: EventStore/replicator-docs#109

Copy link

linear bot commented Feb 9, 2024

@josephcummings josephcummings force-pushed the DEV-270/make-helm-custom-partitioning-consistent-with-custom-transformation branch from ccf6656 to 8c5ebd5 Compare February 12, 2024 09:36
@josephcummings josephcummings self-assigned this Feb 12, 2024
@josephcummings josephcummings changed the title DEV-270 DEV-270 - Improve configuration of transform and partitioner Feb 12, 2024
@josephcummings josephcummings marked this pull request as ready for review February 13, 2024 16:37
@josephcummings josephcummings force-pushed the DEV-270/make-helm-custom-partitioning-consistent-with-custom-transformation branch 3 times, most recently from 29e3db6 to 671369f Compare February 14, 2024 11:19
@josephcummings josephcummings force-pushed the DEV-270/make-helm-custom-partitioning-consistent-with-custom-transformation branch from 671369f to beaef51 Compare February 14, 2024 11:20
@alexeyzimarev alexeyzimarev merged commit e257a18 into master Feb 14, 2024
1 check passed
@alexeyzimarev alexeyzimarev deleted the DEV-270/make-helm-custom-partitioning-consistent-with-custom-transformation branch February 14, 2024 12:00
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