Merge pull request #204 from GameAnalytics/having

Add support for `is_null` filter

Author: danilagamma

.github/dependabot.yml

@@ -6,7 +6,18 @@ updates:
     interval: daily
     time: "04:00"
   open-pull-requests-limit: 10
+  groups:
+    mix-minor-updates:
+      patterns:
+      - "*"
+      update-types:
+      - "minor"
+      - "patch"
 - package-ecosystem: "github-actions"
   directory: "/"
   schedule:
     interval: monthly
+  groups:
+    gh-actions:
+      patterns:
+      - "*"

CODEOWNERS.md

@@ -0,0 +1 @@
+* @GameAnalytics/ga-backend

lib/panoramix/query.ex

@@ -718,6 +718,10 @@ defmodule Panoramix.Query do
     end
   end
 
+  defp build_filter({:is_null, _, [column]}) do
+    {:%{}, [], [type: "null", column: column]}
+  end
+
   defp build_filter({:expression, _, [expression]}) do
     # A math expression, as described in http://druid.io/docs/0.12.1/misc/math-expr
     # We're expecting a string that we're passing on to Druid

test/panoramix_test.exs

@@ -1540,4 +1540,27 @@ defmodule PanoramixTest do
                                   %{"expression" => "foo + 2", "name" => "plus_two", "outputType" => "double", "type" => "expression"}]} =
       query2 |> Panoramix.Query.to_json() |> Jason.decode!()
   end
+
+  test "builds a query with is_null filter" do
+    query =
+      from "table",
+        query_type: "groupBy",
+        filter: is_null(:foo) and not is_null(:bar)
+
+    assert query.filter
+
+    json = Panoramix.Query.to_json(query)
+
+    assert %{
+             "queryType" => "groupBy",
+             "context" => %{"priority" => 0, "timeout" => 120_000},
+             "filter" => %{
+               "type" => "and",
+               "fields" => [
+                 %{"type" => "null", "column" => "foo"},
+                 %{"type" => "not", "field" => %{"type" => "null", "column" => "bar"}}
+               ]
+             }
+           } = Jason.decode!(json)
+  end
 end