Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add --no-extensio-on-workers to citus_dev #77

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

JelteF
Copy link
Contributor

@JelteF JelteF commented Jun 12, 2020

Running is causing problems in v9.3.0, but not in v9.2.4:

./citus_dev make worker-extension-missing --use-ssl --no-extension-on-workers --destroy
WARNING:  42883: function assign_distributed_transaction_id(integer, integer, unknown) does not exist
HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
LOCATION:  ReportResultError, remote_commands.c:343
ERROR:  XX000: failure on connection marked as essential: localhost:9701
LOCATION:  MarkRemoteTransactionFailed, remote_transaction.c:731

The same is true without --use-ssl, but then it fails with an SSL error. I
think this is probably desired though:

./citus_dev make worker-extension-missing --no-extension-on-workers --destroy
ERROR:  08006: connection to the remote node localhost:9701 failed with the following error: server does not support SSL, but SSL was required
LOCATION:  ReportConnectionError, remote_commands.c:289

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

Successfully merging this pull request may close these issues.

1 participant