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.

Sign up for GitHub

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

Jump to bottom

Snowflake loader: region and account configuration fields should be optional #947

Closed
istreeter opened this issue Jun 23, 2022 · 0 comments
Closed

Snowflake loader: region and account configuration fields should be optional #947

istreeter opened this issue Jun 23, 2022 · 0 comments
Milestone
4.1.0

Comments

@istreeter
Copy link
Contributor

Currently there are two alternative ways to configure the Snowflake storage target. Either:

  1. Provide a jdbcHost, which explicitly sets the hostname
  2. ...or provide a account and snowflakeRegion. From these two fields we can derive the hostname.

The problem is, account and snowflakeRegion are currently required fields. So the user needs to set them even if when they provide a jdbcHost. It doesn't make sense, because account and snowflakeRegion should be ignored if jdbcHost is given.
@istreeter istreeter added this to the 4.1.0 milestone Jun 23, 2022
voropaevp added a commit that referenced this issue Jun 24, 2022
@voropaevp
Snowflake loader: region and account configuration fields should be o…
3418529 
…ptional (close #947)
voropaevp added a commit that referenced this issue Jun 24, 2022
@voropaevp
Snowflake loader: region and account configuration fields should be o…
342c086 
…ptional (close #947)
istreeter pushed a commit that referenced this issue Jun 25, 2022
@voropaevp @istreeter
Snowflake loader: region and account configuration fields should be o…
d4aa449 
…ptional (close #947)
istreeter pushed a commit that referenced this issue Jun 25, 2022
@voropaevp @istreeter
Snowflake loader: region and account configuration fields should be o…
654d294 
…ptional (close #947)
istreeter pushed a commit that referenced this issue Jun 25, 2022
@voropaevp @istreeter
Snowflake loader: region and account configuration fields should be o…
8627c41 
…ptional (close #947)
istreeter pushed a commit that referenced this issue Jun 25, 2022
@voropaevp @istreeter
Snowflake loader: region and account configuration fields should be o…
47c338f 
…ptional (close #947)
@spenes spenes closed this as completed in 29498d0 Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.1.0
Development

No branches or pull requests
1 participant
@istreeter