Paths

Table of Contentst

Differential D5034

[services] Rust Integration - Backup - Rust - Add converting functions
ClosedPublic
Actions

Authored by • karol on Sep 2 2022, 3:51 AM.

Tags

None

Referenced Files

	F2094462: D5034.id16487.diff
	Mon, Jun 24, 5:34 AM

	Unknown Object (File)
	Sun, Jun 23, 2:53 AM

	Unknown Object (File)
	Sat, Jun 22, 7:21 PM

	Unknown Object (File)
	Sat, Jun 22, 4:33 PM

	Unknown Object (File)
	Fri, Jun 21, 10:42 AM

	Unknown Object (File)
	Wed, Jun 19, 8:16 PM

	Unknown Object (File)
	Thu, Jun 13, 6:20 AM

	Unknown Object (File)
	Sun, Jun 2, 3:44 AM

View All 87 Files

Subscribers

Details

Reviewers

• jon
tomek
varun
• max

Commits

rCOMM396b990da3e4: [services] Rust Integration - Backup - Rust - Add converting functions

Summary

Depends on D5031

Adding helper functions to convert between rust's String and c++'s char* both ways.

Test Plan

backup builds successfully

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• karol created this revision.Sep 2 2022, 3:51 AM

Herald added a reviewer: • jon. · View Herald TranscriptSep 2 2022, 3:51 AM

Herald added subscribers: • abosh, atul, tomek, ashoat. · View Herald Transcript

• karol edited the summary of this revision. (Show Details)Sep 2 2022, 3:55 AM

• karol edited the test plan for this revision. (Show Details)

• karol added reviewers: tomek, varun, • max.

• karol added a parent revision: D5031: [services] Rust Integration - Backup - Make check_error shared.

• karol added a child revision: D5035: [services] Rust Integration - Backup - Rust - Multiple put clients.Sep 2 2022, 3:59 AM

Harbormaster completed remote builds in B11828: Diff 16233.Sep 2 2022, 4:01 AM

• karol requested review of this revision.Sep 2 2022, 4:01 AM

tomek requested changes to this revision.Sep 5 2022, 7:49 AM

tomek added inline comments.

services/backup/blob_client/src/tools.rs
37 ↗	(On Diff #16233)	We should avoid using `unwrap`

This revision now requires changes to proceed.Sep 5 2022, 7:49 AM

• karol added inline comments.Sep 6 2022, 4:22 AM

services/backup/blob_client/src/tools.rs
37 ↗	(On Diff #16233)	Right, I will fix this.

avoid unwrap

Harbormaster completed remote builds in B11913: Diff 16356.Sep 6 2022, 4:31 AM

tomek accepted this revision.Sep 7 2022, 6:02 AM

tomek added inline comments.

services/backup/blob_client/src/tools.rs
1 ↗	(On Diff #16356)	Do we need this line?
23–25 ↗	(On Diff #16356)	This should be formatted in the diff that introduced this code.
35 ↗	(On Diff #16356)	This line seems to be too long - is it formatted?

This revision is now accepted and ready to land.Sep 7 2022, 6:02 AM

• karol added inline comments.Sep 8 2022, 3:37 AM

services/backup/blob_client/src/tools.rs
1 ↗	(On Diff #16356)	Apparently, no. Weird that `cargo check` didn't catch this.
35 ↗	(On Diff #16356)	will do

rebase

This revision was landed with ongoing or failed builds.Sep 8 2022, 4:10 AM

Closed by commit rCOMM396b990da3e4: [services] Rust Integration - Backup - Rust - Add converting functions (authored by • karol). · Explain Why

This revision was automatically updated to reflect the committed changes.

• karol added a commit: rCOMM396b990da3e4: [services] Rust Integration - Backup - Rust - Add converting functions.

Harbormaster completed remote builds in B12005: Diff 16460.Sep 8 2022, 4:35 AM

Revision Contents
Changeset List

Path

Size

services/

backup/

blob_client/

src/

26 lines

Diff 16487

services/backup/blob_client/src/tools.rs

Loading...