-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Add SynchronizedItemStreamWriter #842
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
Comments
Dimitrios Liapis commented Mahmoud Ben Hassine Would that request make sense? If so I could take a look |
Mahmoud Ben Hassine commented Yes, this feature has been discussed on gitter and gd-estrepetov suggested to open a PR for it. This make sense when used with the Json item writer for example (as explained here) and is a nice addition IMO to complement the But everyone is welcome to contribute! |
Dimitrios Liapis commented ah sure thanks, I wasn't aware of the gitter channel of course gd-estrepetov has precedence :) gd-estrepetov - have you started working on this by any chance? |
Dimitrios Liapis commented Just realised that there's been some time since the last update on this issue so I've submitted a PR In case gd-estrepetov has already started on this I can retract this PR |
gd-estrepetov commented Dimitrios Liapis, I am busy at my daily job, so I am good with your PR. You can take this task. |
Dimitrios Liapis commented gd-estrepetov many thanks |
Resolved with #665 . |
gd-estrepetov opened BATCH-2762 and commented
This could be useful especially for writers like JsonFileItemWriter.
Affects: 4.1.0
Referenced from: pull request #665
The text was updated successfully, but these errors were encountered: