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

Fix for issue #40220 #52959

Merged
merged 11 commits into from Aug 9, 2023
Merged

Fix for issue #40220 #52959

merged 11 commits into from Aug 9, 2023

Conversation

jiyoungyoooo
Copy link
Contributor

@jiyoungyoooo jiyoungyoooo commented Aug 3, 2023

improve error message for table function remote

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

Improve error message for table function remote. Closes #40220.

improve error message for table function remote
@kssenii kssenii self-assigned this Aug 3, 2023
@kssenii kssenii added the can be tested Allows running workflows for external contributors label Aug 3, 2023
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-improvement Pull request with some product improvements label Aug 3, 2023
@robot-ch-test-poll3
Copy link
Contributor

robot-ch-test-poll3 commented Aug 3, 2023

This is an automated comment for commit 47cccee with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟡 pending

Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR🟡 pending
Mergeable CheckChecks if all other necessary checks are successful🟢 success

src/TableFunctions/TableFunctionRemote.cpp Outdated Show resolved Hide resolved
src/TableFunctions/TableFunctionRemote.cpp Outdated Show resolved Hide resolved
@clickhouse-ci clickhouse-ci bot added the manual approve Manual approve required to run CI label Aug 5, 2023
@kssenii
Copy link
Member

kssenii commented Aug 9, 2023

Integration tests (asan) [1/6]

test_zookeeper_config

E           subprocess.TimeoutExpired: Command '['docker-compose', '--env-file', '/ClickHouse/tests/integration/test_zookeeper_config/_instances_0/.env', '--project-name', 'roottestzookeeperconfig', '--file', '/ClickHouse/tests/integration/test_zookeeper_config/_instances_0/node1/docker-compose.yml', '--file', '/compose/docker_compose_keeper.yml', '--file', '/ClickHouse/tests/integration/test_zookeeper_config/_instances_0/node2/docker-compose.yml', '--file', '/ClickHouse/tests/integration/test_zookeeper_config/_instances_0/node3/docker-compose.yml', 'up', '-d', '--no-recreate']' timed out after 300 seconds

Integration tests (tsan) [5/6]

test_storage_mongodb

E               ERROR: for mongo2  Cannot start service mongo2: driver failed programming external connectivity on endpoint rootteststoragemongodb_mongo2_1 (4ee552a231ca81d5ded76bdf3374a628cc2ca6eb074cee4515577efe741e7b85): Bind for 0.0.0.0:63491 failed: port is already allocated
E               compose.cli.main.exit_with_metrics: Encountered errors while bringing up the project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
can be tested Allows running workflows for external contributors manual approve Manual approve required to run CI pr-improvement Pull request with some product improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong error message from table function remote
3 participants