Github user asfgit closed the pull request at:
https://github.com/apache/flink/pull/5021
---
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151188168
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151187261
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151184658
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151184079
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151184645
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@@
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151184384
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151183892
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151183957
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151183730
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151183695
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151183131
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151182879
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151182277
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/main/java/org/apache/flink/queryablestate/client/proxy/KvStateClientProxyHandler.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151179694
--- Diff:
flink-queryable-state/flink-queryable-state-runtime/src/test/java/org/apache/flink/queryablestate/itcases/AbstractQueryableStateTestBase.java
---
@
Github user zentol commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151179388
--- Diff:
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/network/AbstractServerHandler.java
---
@@ -26
Github user kl0u commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151162640
--- Diff:
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/network/AbstractServerHandler.java
---
@@ -262,
Github user aljoscha commented on a diff in the pull request:
https://github.com/apache/flink/pull/5021#discussion_r151160873
--- Diff:
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/network/AbstractServerHandler.java
---
@@ -
GitHub user kl0u opened a pull request:
https://github.com/apache/flink/pull/5021
[FLINK-8063][QS] QS client does not retry when an UnknownKvStateLocation.
## What is the purpose of the change
Fix bug in QS client. Before if the client submitted a query with invalid
queryab
19 matches
Mail list logo