Ralph Jennings

Results 26 comments of Ralph Jennings

> > configBuilder.addArg("--default-time-zone=UTC"); I had to use `.addArg("--default-time-zone=+0:00")` as it wouldn't recognize "GMT" or "UTC" as valid timezones either. > interesting. Perhaps `DBConfigurationBuilder` should have a `setTimeZone(java.util.TimeZone tz)` method? With...

This could be solved using `@CustomParam` annotation if #667 gets pulled in. The caveat here being that interceptors are run again on every retry, while param encoders are only run...

I kinda think `RetryableException` was a mistake. Why not just have the retryer take the exception ("retryable" or otherwise) and let it determine whether or not the exception truly is...

@RiddhiKhatri and @Shapoval1van: if you're asking a question, stackoverflow might be better. If you are reporting a bug, then a reproducible example would help.

Would it make more sense to ignore all non-default methods *not* annotated with `@RequestLine` (or the equivalent in the `Contract` that is being used) and throw `UnsupportedOperationException` if those non-implemented...

IMO, the cleaner solution would be to make multiple interfaces (your concrete class on the server side can implement them both). The one for Feign, the other for your queue....

> ```java > Set artifactoryLoggers = new HashSet(Arrays.asList("org.apache.http", "groovyx.net.http")); > for(String log:artifactoryLoggers) { > ch.qos.logback.classic.Logger artLogger = (ch.qos.logback.classic.Logger) org.slf4j.LoggerFactory.getLogger(log); > artLogger.setLevel(ch.qos.logback.classic.Level.INFO); > artLogger.setAdditive(false); > } > ``` @Dougnlizt Is there...

@tyler2cr the "" is just how JUnit describes an empty string in the `assertNull` error message.

There should definitely be some way to know what 'pt' is going to become before it is used. Right now it depends on the order of the internal UNITS declaration...

There are 2 separate temperature references. One is absolute temperature (ex: "tempK") which you cannot divide with. The other is relative degrees (ex: "degK") which you should be able to...