sreeram26 commented on a change in pull request #1898:
URL: https://github.com/apache/hudi/pull/1898#discussion_r464681579
##########
File path:
hudi-utilities/src/main/java/org/apache/hudi/utilities/deltastreamer/HoodieDeltaStreamer.java
##########
@@ -165,6 +168,12 @@ public Operation convert(String value) throws
ParameterException {
}
}
+ private static final class VoidSplitter implements IParameterSplitter {
Review comment:
I was a little skeptical about making this change apply for all CLIs.
But on second thought, looked at all documentation and README files all of them
have `--hoodie-conf A --hoodie-conf B` way so no one in theory should be
sending `--hoodie-conf A,B` in their usecases. So will move it to common and
apply this change for all of the classes using hoodie-conf
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
