hermannpencole / nifi-config

Automatically deploy and configure Template on Nifi
Apache License 2.0
55 stars 39 forks source link

The new timeout configuraton is not working: #14

Closed bbossola-tesco closed 6 years ago

bbossola-tesco commented 7 years ago

From the code:

Integer connectionTimeout = cmd.hasOption("connectionTimeout") ? Integer.valueOf(cmd.getOptionValue("connectionTimeout")) :10000;
Integer readTimeout = cmd.hasOption("connectionTimeout") ? Integer.valueOf(cmd.getOptionValue("readTimeout")) :10000;
Integer writeTimeout = cmd.hasOption("connectionTimeout") ? Integer.valueOf(cmd.getOptionValue("writeTimeout")) :10000;

Clearly a cut&paste error: only by specifying "connectionTimeout" the other values will be loaded

bbossola-tesco commented 7 years ago

PR here: https://github.com/hermannpencole/nifi-config/pull/15

hermannpencole commented 7 years ago

You are quite right ; it's clearly a cut&paste error: I am not so glad about that