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.

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

Add cortex_search_service to proto def #213

Merged
merged 1 commit into from
Nov 27, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 12 additions & 2 deletions semantic_model_generator/protos/semantic_model.proto
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,10 @@ message Column {
// Retrieved literals of this column.
repeated RetrievalResult retrieved_literals = 11 [(optional) = true];

// Name of a Cortex Search Service configured on this column.
string cortex_search_service_name = 12 [(optional) = true];
// A Cortex Search Service configured on this column to retrieve literals.
string cortex_search_service_name = 12
[(optional) = true, deprecated = true];
CortexSearchService cortex_search_service = 13 [(optional) = true];
}

// Dimension columns contain categorical values (e.g. state, user_type,
Expand Down Expand Up @@ -109,6 +111,14 @@ message Dimension {
string cortex_search_service_name = 9 [(optional) = true];
}

// Fully qualified Cortex Search Service name.
message CortexSearchService {
string database = 1 [(optional) = true];
string schema = 2 [(optional) = true];
string service = 3;
string literal_column = 4 [(optional) = true];
}

// Time dimension columns contain time values (e.g. sale_date, created_at,
// year). NOTE: If modifying this protobuf, make appropriate changes in
// to_column_format() of snowpilot/semantic_context/utils/utils.py.
Expand Down
Loading
Loading