Skip to content

Commit

Permalink
fix: typo in the error message if a column is not found
Browse files Browse the repository at this point in the history
  • Loading branch information
Taepper committed Aug 7, 2024
1 parent d1c32e5 commit b3147d5
Show file tree
Hide file tree
Showing 7 changed files with 7 additions and 8 deletions.
3 changes: 1 addition & 2 deletions src/silo/query_engine/filter_expressions/bool_equals.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> BoolEquals::compile(
) const {
CHECK_SILO_QUERY(
database_partition.columns.bool_columns.contains(column_name),
// TODO
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

const auto& bool_column = database_partition.columns.bool_columns.at(column_name);
Expand Down
2 changes: 1 addition & 1 deletion src/silo/query_engine/filter_expressions/date_between.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ std::unique_ptr<operators::Operator> DateBetween::compile(
) const {
CHECK_SILO_QUERY(
database_partition.columns.date_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

const auto& date_column = database_partition.columns.date_columns.at(column_name);
Expand Down
2 changes: 1 addition & 1 deletion src/silo/query_engine/filter_expressions/float_equals.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> FloatEquals::compile(
) const {
CHECK_SILO_QUERY(
database_partition.columns.float_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

const auto& float_column = database_partition.columns.float_columns.at(column_name);
Expand Down
2 changes: 1 addition & 1 deletion src/silo/query_engine/filter_expressions/int_between.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> IntBetween::compile(
) const {
CHECK_SILO_QUERY(
database_partition.columns.int_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

const auto& int_column = database_partition.columns.int_columns.at(column_name);
Expand Down
2 changes: 1 addition & 1 deletion src/silo/query_engine/filter_expressions/int_equals.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> IntEquals::compile(
) const {
CHECK_SILO_QUERY(
database_partition.columns.int_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

const auto& int_column = database_partition.columns.int_columns.at(column_name);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> PangoLineageFilter::com
) const {
CHECK_SILO_QUERY(
database_partition.columns.pango_lineage_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

std::string lineage_all_upper = lineage;
Expand Down
2 changes: 1 addition & 1 deletion src/silo/query_engine/filter_expressions/string_equals.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ std::unique_ptr<silo::query_engine::operators::Operator> StringEquals::compile(
CHECK_SILO_QUERY(
database_partition.columns.string_columns.contains(column_name) ||
database_partition.columns.indexed_string_columns.contains(column_name),
fmt::format("the database does not contain the column '{}'", column_name)
fmt::format("The database does not contain the column '{}'", column_name)
);

if (database_partition.columns.indexed_string_columns.contains(column_name)) {
Expand Down

0 comments on commit b3147d5

Please sign in to comment.