BUG: get_indexer_not_unique inconsistent return types vs get_indexer pandas-dev#16819

Author: ri938

doc/source/whatsnew/v0.21.0.txt

@@ -91,6 +91,7 @@ Performance Improvements
 
 Bug Fixes
 ~~~~~~~~~
+- Bug in get_indexer_non_unique inconsistent return type with get_indexer (:issue:`16819`)
 
 Conversion
 ^^^^^^^^^^

pandas/core/indexes/base.py

@@ -2704,7 +2704,7 @@ def get_indexer_non_unique(self, target):
             tgt_values = target._values
 
         indexer, missing = self._engine.get_indexer_non_unique(tgt_values)
-        return Index(indexer), missing
+        return indexer, missing
 
     def get_indexer_for(self, target, **kwargs):
         """

pandas/tests/indexes/test_base.py

@@ -1131,6 +1131,17 @@ def test_get_indexer_strings(self):
         with pytest.raises(TypeError):
             idx.get_indexer(['a', 'b', 'c', 'd'], method='pad', tolerance=2)
 
+    def test_get_indexer_consistency(self):
+        # See GH 16819
+        for name, index in self.indices.items():
+            indexer = index.get_indexer(index[0:2])
+            assert isinstance(indexer, np.ndarray)
+            assert indexer.dtype == np.intp
+
+            indexer, _ = index.get_indexer_non_unique(index[0:2])
+            assert isinstance(indexer, np.ndarray)
+            assert indexer.dtype == np.intp
+
     def test_get_loc(self):
         idx = pd.Index([0, 1, 2])
         all_methods = [None, 'pad', 'backfill', 'nearest']

pandas/tests/indexes/test_category.py

@@ -386,8 +386,7 @@ def test_reindexing(self):
             expected = oidx.get_indexer_non_unique(finder)[0]
 
             actual = ci.get_indexer(finder)
-            tm.assert_numpy_array_equal(
-                expected.values, actual, check_dtype=False)
+            tm.assert_numpy_array_equal(expected, actual, check_dtype=True)
 
     def test_reindex_dtype(self):
         c = CategoricalIndex(['a', 'b', 'c', 'a'])