Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/6028#discussion_r189015635
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@@ -225,7 +225,9 @@ private void executeActionAsync(
return location;
} else {
- return FutureUtils.completedExceptionally(new
UnknownLocationException("Could not contact the state location oracle to
retrieve the state location."));
+ return FutureUtils.completedExceptionally(
+ new UnknownLocationException("Could not contact
the state location oracle to retrieve the state location for state="
+ + queryableStateName + " of job=" +
jobId + ", the caused reason maybe the state is not ready or there is no job
exists."));
--- End diff --
This message is pretty verbose and it contains grammatical errors. If it
were to write this, it should be something like:
```
Could not contact the state location oracle to retrieve the location for
state=QSName of job= JobID. The reason can be that the state is not ready or
that that does not exist.
```
But before putting this in, it would be helpful if @florianschmidt1994
commented on what he thinks, as he is the one that opened the issue.
---
