Skip to content

Commit

Permalink
snowflake-api: make request-id uuid4
Browse files Browse the repository at this point in the history
  • Loading branch information
andrusha committed Aug 18, 2023
1 parent 792bfc7 commit 806133a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion snowflake-api/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ log = "0.4"
serde_json = "1"
serde = { version = "1", features = ["derive"] }
url = "2"
uuid = { version = "1.4", features = ["v1", "v4"] }
uuid = { version = "1.4", features = ["v4"] }
arrow = "42"
base64 = "0.21"
regex = "1"
Expand Down
4 changes: 2 additions & 2 deletions snowflake-api/src/connection.rs
Original file line number Diff line number Diff line change
Expand Up @@ -98,12 +98,12 @@ impl Connection {
) -> Result<R, ConnectionError> {
let context = query_type.query_context();

// todo: increment subsequent request ids (on retry?)
let request_id = Uuid::now_v1(&[0, 0, 0, 0, 0, 0]);
let request_id = Uuid::new_v4();
let request_guid = Uuid::new_v4();
let (client_start_time, _nanos) = request_id.get_timestamp().unwrap().to_unix();

let client_start_time = client_start_time.to_string();
// fixme: update uuid's on the retry
let request_id = request_id.to_string();
let request_guid = request_guid.to_string();

Expand Down

0 comments on commit 806133a

Please sign in to comment.