Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

checkWatch networkReadMillis < cacheWatchSeconds ? #451

Open
ecolinet opened this issue Jul 29, 2021 · 1 comment
Open

checkWatch networkReadMillis < cacheWatchSeconds ? #451

ecolinet opened this issue Jul 29, 2021 · 1 comment

Comments

@ecolinet
Copy link

ecolinet commented Jul 29, 2021

Hi,

I don't understand why the networkReadTimeout is enforced to be higher than the cacheWatch value. Shouldn't it be the opposite ?

protected static void checkWatch(int networkReadMillis, int cacheWatchSeconds) {
if (networkReadMillis <= TimeUnit.SECONDS.toMillis(cacheWatchSeconds)) {
throw new IllegalArgumentException("Cache watchInterval="+ cacheWatchSeconds + "sec >= networkClientReadTimeout="
+ networkReadMillis + "ms. It can cause issues");
}
}

Thanks,
Eric

@ElfoLiNk
Copy link

I think #453 will address this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants