Paths

Table of Contentst

[services] add consts for secondary index names
ClosedPublic
Actions

Authored by varun on Jul 26 2022, 9:41 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 9, 9:00 AM

	Unknown Object (File)
	Tue, Nov 5, 10:01 PM

	Unknown Object (File)
	Tue, Nov 5, 9:45 PM

	Unknown Object (File)
	Tue, Nov 5, 6:58 PM

	Unknown Object (File)
	Sun, Oct 27, 6:19 PM

	Unknown Object (File)
	Sun, Oct 27, 6:19 PM

	Unknown Object (File)
	Sun, Oct 27, 6:19 PM

	Unknown Object (File)
	Sun, Oct 27, 6:16 PM

Subscribers

Details

Reviewers

tomek
• karol

Commits

rCOMMf654bf51591b: [services] add consts for secondary index names

Summary

follow-up to the terraform diff that introduced two new GSIs. we will have to query by index name so we should make the names consts.

depends on D4647

Test Plan

cargo build

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Jul 26 2022, 9:41 PM

Herald added subscribers: • abosh, atul, • adrian, ashoat. · View Herald TranscriptJul 26 2022, 9:41 PM

Harbormaster completed remote builds in B10902: Diff 15001.Jul 26 2022, 9:53 PM

varun requested review of this revision.Jul 26 2022, 9:53 PM

tomek accepted this revision.Jul 27 2022, 4:16 AM

This revision is now accepted and ready to land.Jul 27 2022, 4:16 AM

varun added a child revision: D4656: [services] Helper function to parse userID from AttributeValue.Jul 27 2022, 1:36 PM

Closed by commit rCOMMf654bf51591b: [services] add consts for secondary index names (authored by varun). · Explain WhyJul 28 2022, 11:45 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMf654bf51591b: [services] add consts for secondary index names.

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

constants.rs

2 lines

Diff 15067

View Options

services/identity/src/constants.rs

[services] add consts for secondary index namesClosedPublicActions