Merge pull request #21 from Recruitee/change-campaign-id-and-product-id-field-types

Change campaign_id and product_id field types

Author: rdk08

lib/plausible/event/clickhouse_schema.ex

@@ -15,8 +15,6 @@ defmodule Plausible.ClickhouseEvent do
     field :site_id, :integer
     field :page_id, :integer
     field :job_id, :integer
-    field :campaign_id, :integer
-    field :product_id, :integer
     field :timestamp, :naive_datetime
 
     field :referrer, :string, default: ""
@@ -26,6 +24,8 @@ defmodule Plausible.ClickhouseEvent do
     field :utm_campaign, :string, default: ""
     field :utm_content, :string, default: ""
     field :utm_term, :string, default: ""
+    field :campaign_id, :string, default: ""
+    field :product_id, :string, default: ""
 
     field :country_code, :string, default: ""
     field :subdivision1_code, :string, default: ""

lib/plausible/session/clickhouse_schema.ex

@@ -10,8 +10,6 @@ defmodule Plausible.ClickhouseSession do
     field :session_id, :integer
     field :company_id, :integer
     field :site_id, :integer
-    field :campaign_id, :integer
-    field :product_id, :integer
 
     field :start, :naive_datetime
     field :duration, :integer
@@ -29,6 +27,8 @@ defmodule Plausible.ClickhouseSession do
     field :utm_term, :string
     field :referrer, :string
     field :referrer_source, :string
+    field :campaign_id, :string
+    field :product_id, :string
 
     field :country_code, :string, default: ""
     field :subdivision1_code, :string, default: ""

priv/clickhouse_repo/migrations/20221130162330_remove_campaign_id.exs

@@ -0,0 +1,13 @@
+defmodule Plausible.ClickhouseRepo.Migrations.RemoveCampaignId do
+  use Ecto.Migration
+
+  def change do
+    alter table(:events) do
+      remove :campaign_id
+    end
+
+    alter table(:sessions) do
+      remove :campaign_id
+    end
+  end
+end

priv/clickhouse_repo/migrations/20221130162430_remove_product_id.exs

@@ -0,0 +1,13 @@
+defmodule Plausible.ClickhouseRepo.Migrations.RemoveProductId do
+  use Ecto.Migration
+
+  def change do
+    alter table(:events) do
+      remove :product_id
+    end
+
+    alter table(:sessions) do
+      remove :product_id
+    end
+  end
+end

priv/clickhouse_repo/migrations/20221130174130_recreate_campaign_id_and_product_id.exs

@@ -0,0 +1,15 @@
+defmodule Plausible.ClickhouseRepo.Migrations.RecreateCampaignIdAndProductId do
+  use Ecto.Migration
+
+  def change do
+    alter table(:events) do
+      add(:campaign_id, :string)
+      add(:product_id, :string)
+    end
+
+    alter table(:sessions) do
+      add(:campaign_id, :string)
+      add(:product_id, :string)
+    end
+  end
+end