Fixed error with --pg-stat-statements and --min-calls - fixes #53

CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.5.5 (unreleased)
+
+- Fixed error with `--pg-stat-statements` and `--min-calls`
+
 ## 0.5.4 (2024-04-03)
 
 - Fixed issue with processing over 500 query fingerprints (introduced in 0.5.3)

lib/dexter/client.rb

@@ -64,6 +64,7 @@ def parse_args(args)
 
  o.separator "Processing options:"
  o.integer "--interval", "time to wait between processing queries, in seconds", default: 60
+ # TODO change to integer in 0.6.0
  o.float "--min-calls", "only process queries that have been called a certain number of times", default: 0
  o.float "--min-time", "only process queries that have consumed a certain amount of DB time, in minutes", default: 0
  o.separator ""

lib/dexter/indexer.rb

@@ -682,7 +682,7 @@ def stat_statements
  ORDER BY
  1
  SQL
- execute(sql, params: [@min_time * 60000, @min_calls]).map { |q| q["query"] }
+ execute(sql, params: [@min_time * 60000, @min_calls.to_i]).map { |q| q["query"] }
  end
 
  def with_advisory_lock

test/input_test.rb

@@ -27,6 +27,7 @@ def test_pg_stat_statements
  execute("SELECT pg_stat_statements_reset()")
  execute("SELECT * FROM posts WHERE id = 1")
  assert_dexter_output "Index found: public.posts (id)", ["--pg-stat-statements"]
+ assert_dexter_output "Index found: public.posts (id)", ["--pg-stat-statements", "--min-calls", "1"]
  end
 
  def test_no_source