Skip to content
This repository has been archived by the owner on Sep 26, 2023. It is now read-only.

Use non-AWS endpoints in host inference #131

Closed
chuwy opened this issue Feb 25, 2021 · 0 comments
Closed

Use non-AWS endpoints in host inference #131

chuwy opened this issue Feb 25, 2021 · 0 comments
Assignees
Milestone

Comments

@chuwy
Copy link
Contributor

chuwy commented Feb 25, 2021

In

val host = config.jdbcHost match {
case Some(overrideHost) => overrideHost
case None =>
if (config.snowflakeRegion == "us-west-2")
s"${config.account}.snowflakecomputing.com"
else if (regionsWithoutAwsSubdomain.contains(config.snowflakeRegion))
s"${config.account}.${config.snowflakeRegion}.snowflakecomputing.com"
else
s"${config.account}.${config.snowflakeRegion}.aws.snowflakecomputing.com"
}
we check if regional endpoint doesn't contain aws part and add it to all regions otherwise.

For non-AWS regions we can end up west-europe.azure.aws.

Since #59 we can overwrite the host, but we can improve inference to use it less often.

@chuwy chuwy self-assigned this Feb 25, 2021
@dilyand dilyand changed the title Change endpoint inferernce to use non-aws endpoints by default Loader: use non-AWS endpoints in host inference Jun 25, 2021
@dilyand dilyand added this to the 0.8.1 milestone Jun 25, 2021
@dilyand dilyand changed the title Loader: use non-AWS endpoints in host inference Use non-AWS endpoints in host inference Jun 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants