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

thread name for executor #515

Closed
wants to merge 2 commits into from
Closed

thread name for executor #515

wants to merge 2 commits into from

Conversation

sdyura
Copy link

@sdyura sdyura commented Aug 3, 2023

No description provided.

@cla-bot
Copy link

cla-bot bot commented Aug 3, 2023

Fresh avocado detected! Welcome and thank you for your contribution @sdyura. My avocado-loving overlords have decreed a signed CLA is required for PRs. Please see https://github.com/arangodb/arangodb/blob/devel/CONTRIBUTING.md file to learn more or ask @cw00dw0rd if you have issues.

@cla-bot
Copy link

cla-bot bot commented Aug 3, 2023

Fresh avocado detected! Welcome and thank you for your contribution @sdyura. My avocado-loving overlords have decreed a signed CLA is required for PRs. Please see https://github.com/arangodb/arangodb/blob/devel/CONTRIBUTING.md file to learn more or ask @cw00dw0rd if you have issues.

Copy link

sonarcloud bot commented Jul 29, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@rashtao
Copy link
Collaborator

rashtao commented Sep 20, 2024

Closing since not relevant anymore. Threads used in the HttpConnection are prefixed with adb-http-, see

vertxToUse.runOnContext(e -> Thread.currentThread().setName("adb-http-" + THREAD_COUNT.getAndIncrement()));

For VST this is not relevant anymore, since such protocol has been removed from ArangoDB since version 3.12.

@rashtao rashtao closed this Sep 20, 2024
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.

2 participants