Skip to content

Commit

Permalink
[CALCITE-1399] Make the jcommander SerializationConverter public
Browse files Browse the repository at this point in the history
When this class is private, JCommander fails because it
cannot instantiate the implementation and fails the program.
  • Loading branch information
joshelser committed Sep 28, 2016
1 parent 514d262 commit c9ec5c8
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,9 @@ public static void main(String[] args) {
}

/**
* Converter from String to Serialization.
* Converter from String to Serialization. Must be public for JCommander.
*/
private static class SerializationConverter implements IStringConverter<Serialization> {
public static class SerializationConverter implements IStringConverter<Serialization> {
@Override public Serialization convert(String value) {
return Serialization.valueOf(value.toUpperCase());
}
Expand Down

0 comments on commit c9ec5c8

Please sign in to comment.