Skip to content

JAVA-629: Don't use connection timeout for unrelated internal queries.#426

Merged
olim7t merged 1 commit into
2.0from
java629
Aug 28, 2015
Merged

JAVA-629: Don't use connection timeout for unrelated internal queries.#426
olim7t merged 1 commit into
2.0from
java629

Conversation

@adutra
Copy link
Copy Markdown
Contributor

@adutra adutra commented Aug 6, 2015

No description provided.

@tolbertam
Copy link
Copy Markdown
Contributor

looks good, runs cleanly in CI, 👍

@olim7t
Copy link
Copy Markdown
Contributor

olim7t commented Aug 24, 2015

Looks good to me.

@olim7t
Copy link
Copy Markdown
Contributor

olim7t commented Aug 24, 2015

The connect timeout is still used in Cluster#connect, but that method gets refactored in #427 and does not use it anymore.

olim7t added a commit that referenced this pull request Aug 28, 2015
JAVA-629: Don't use connection timeout for unrelated internal queries.
@olim7t olim7t merged commit 13c840d into 2.0 Aug 28, 2015
@olim7t olim7t deleted the java629 branch August 28, 2015 13:14
Sfurti-yb pushed a commit to yugabyte/cassandra-java-driver that referenced this pull request Dec 8, 2023
The problem: when our receive buffer fills and conversation demand drops to zero and last item is `CommandComplete` we never request more frames and will never get `ReadyForQuery`

Signed-off-by: Mark Paluch <mpaluch@vmware.com>

[closes apache#401][apache#426]
Sfurti-yb pushed a commit to yugabyte/cassandra-java-driver that referenced this pull request Dec 8, 2023
Remove windowUntil(…) to avoid miscalculation of demand in long-running streams. Refine discardOnCancel operator calls to properly guard simple queries.

[apache#401][apache#426]

Signed-off-by: Mark Paluch <mpaluch@vmware.com>
Sfurti-yb pushed a commit to yugabyte/cassandra-java-driver that referenced this pull request Dec 8, 2023
[apache#401][apache#426]

Signed-off-by: Mark Paluch <mpaluch@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants