From 22d1f2a0c09bed8b77983ed9aa271fbd05070857 Mon Sep 17 00:00:00 2001
From: Gian Merlino <gianmerlino@gmail.com>
Date: Thu, 14 Feb 2013 21:07:17 -0800
Subject: [PATCH] IndexMergerTest: Empty column test

---
 .../druid/index/v1/IndexMergerTest.java       | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/server/src/test/java/com/metamx/druid/index/v1/IndexMergerTest.java b/server/src/test/java/com/metamx/druid/index/v1/IndexMergerTest.java
index 097762106a2..86d03f6dd76 100644
--- a/server/src/test/java/com/metamx/druid/index/v1/IndexMergerTest.java
+++ b/server/src/test/java/com/metamx/druid/index/v1/IndexMergerTest.java
@@ -19,6 +19,7 @@
 
 package com.metamx.druid.index.v1;
 
+import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Lists;
 import com.google.common.io.Files;
@@ -111,4 +112,33 @@ public class IndexMergerTest
       FileUtils.deleteQuietly(mergedDir);
     }
   }
+
+  @Test
+  public void testPersistEmptyColumn() throws Exception
+  {
+    final IncrementalIndex toPersist = new IncrementalIndex(0L, QueryGranularity.NONE, new AggregatorFactory[]{});
+    final File tmpDir = Files.createTempDir();
+
+    try {
+      toPersist.add(
+          new MapBasedInputRow(
+              1L,
+              ImmutableList.of("dim1", "dim2"),
+              ImmutableMap.<String, Object>of("dim1", ImmutableList.of(), "dim2", "foo")
+          )
+      );
+
+      final QueryableIndex merged = IndexIO.loadIndex(
+          IndexMerger.persist(toPersist, tmpDir)
+      );
+
+      Assert.assertEquals(1, merged.getTimeColumn().getLength());
+      Assert.assertEquals(ImmutableList.of("dim1", "dim2"), ImmutableList.copyOf(merged.getAvailableDimensions()));
+      Assert.assertEquals(null, merged.getColumn("dim1"));
+    } finally {
+      FileUtils.deleteQuietly(tmpDir);
+    }
+
+
+  }
 }