Skip to content
This repository was archived by the owner on Nov 30, 2022. It is now read-only.

Commit

Permalink
Merge pull request #350 from sodadata/209-rm-third-param
Browse files Browse the repository at this point in the history
#209 removed 3rd param
  • Loading branch information
vijaykiran authored May 26, 2022
2 parents 9818ece + 9b0cecf commit 6056777
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion core/sodasql/scan/scan.py
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ def _query_aggregations(self):
if scan_column.is_missing_enabled:
metric_indices['non_missing'] = len(measurements)
if scan_column.non_missing_condition:
fields.append(dialect.sql_expr_count_conditional(scan_column.non_missing_condition, column_name, ))
fields.append(dialect.sql_expr_count_conditional(scan_column.non_missing_condition, column_name))
else:
fields.append(dialect.sql_expr_count(scan_column.qualified_column_name))
measurements.append(Measurement(Metric.VALUES_COUNT, column_name))
Expand Down

0 comments on commit 6056777

Please sign in to comment.