Skip to content

Commit

Permalink
add missing columns to query executor sel stmt
Browse files Browse the repository at this point in the history
Signed-off-by: Isabella Basso do Amaral <[email protected]>
  • Loading branch information
isinyaaa committed Oct 2, 2024
1 parent b951b0a commit 31e44df
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ml_metadata/metadata_store/postgresql_query_executor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -903,7 +903,7 @@ absl::Status PostgreSQLQueryExecutor::ListNodeIDsUsingOptions(
sql_gen_status.message()));
}
sql_query = absl::Substitute(
"SELECT distinct $0.id, $0.create_time_since_epoch FROM $1 WHERE $2 "
"SELECT distinct $0.id, $0.create_time_since_epoch, $0.last_update_time_since_epoch FROM $1 WHERE $2 "
"AND ",
*node_table_alias,
// TODO(b/257334039): remove query_version-conditional logic
Expand Down
2 changes: 1 addition & 1 deletion ml_metadata/metadata_store/query_config_executor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -844,7 +844,7 @@ absl::Status QueryConfigExecutor::ListNodeIDsUsingOptions(
sql_gen_status.message()));
}
sql_query = absl::Substitute(
"SELECT distinct $0.`id` FROM $1 WHERE $2 AND ", *node_table_alias,
"SELECT distinct $0.`id`, $0.`create_time_since_epoch`, $0.`last_update_time_since_epoch` FROM $1 WHERE $2 AND ", *node_table_alias,
// TODO(b/257334039): remove query_version-conditional logic
query_builder.GetFromClause(query_version),
query_builder.GetWhereClause());
Expand Down

0 comments on commit 31e44df

Please sign in to comment.