Skip to content
This repository was archived by the owner on Oct 18, 2021. It is now read-only.

Conversation

@Codelone
Copy link
Contributor

meta client connection , 1 second will often timeout, Increase to 6 seconds。

@Codelone Codelone changed the title Liyanl dev Increasemeta client connection timeout。 Sep 30, 2021
@Codelone Codelone changed the title Increasemeta client connection timeout。 Increase meta client connection timeout。 Sep 30, 2021
}

private val metaClient = new MetaClient(address.asJava)
private val metaClient = new MetaClient(address.asJava,6000,3,3)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please use the timeout and retry parameter.


val metaAddress = addresses.map(address => new HostAddress(address._1, address._2)).asJava
val client = new MetaClient(metaAddress)
val client = new MetaClient(metaAddress,6000,3,3)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the timeout and retry number should be parameter

private val DEFAULT_EXECUTION_RETRY_SIZE = 3
val metaAddress = addresses.map(address => new HostAddress(address._1, address._2)).asJava
val client = new MetaClient(metaAddress)
val client = new MetaClient(metaAddress,DEFAULT_TIMEOUT_MS,DEFAULT_CONNECTION_RETRY_SIZE,DEFAULT_EXECUTION_RETRY_SIZE)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should get timeout and retry parameter from NebulaConnectionConfig

@Nicole00
Copy link
Contributor

Thanks for your contribution for Nebula-Exchange and Nebula-Spark-Connector, now this repo has been split into three different repos, could you please submit two pr to repo https://github.com/vesoft-inc/nebula-exchang and repo https://github.com/vesoft-inc/nebula-spark-connector.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants