Replace _has_coplex_internals #29227

Author: Aivengoe

pandas/_libs/reduction.pyx

@@ -5,6 +5,8 @@ from cpython.ref cimport Py_INCREF
 
 from libc.stdlib cimport malloc, free
 
+from pandas.core.dtypes.generic import ABCMultiIndex
+
 import numpy as np
 cimport numpy as cnp
 from numpy cimport (ndarray,
@@ -522,7 +524,7 @@ def apply_frame_axis0(object frame, object f, object names,
         object piece
         dict item_cache
 
-    if frame.index._has_complex_internals:
+    if isinstance(frame.index, ABCMultiIndex):
         raise InvalidApply('Cannot modify frame index internals')
 
     results = []
@@ -653,7 +655,7 @@ def compute_reduction(arr, f, axis=0, dummy=None, labels=None):
 
     if labels is not None:
         # Caller is responsible for ensuring we don't have MultiIndex
-        assert not labels._has_complex_internals
+        assert not isinstance(frame.index, ABCMultiIndex),('MultiIndex type')
 
         # pass as an ndarray/ExtensionArray
         labels = labels._values

pandas/core/apply.py

@@ -5,6 +5,8 @@
 from pandas._libs import reduction as libreduction
 from pandas.util._decorators import cache_readonly
 
+from pandas.core.dtypes.generic import ABCMultiIndex
+
 from pandas.core.dtypes.common import (
     is_dict_like,
     is_extension_type,
@@ -233,7 +235,7 @@ def apply_standard(self):
             and not self.dtypes.apply(is_extension_type).any()
             # Disallow complex_internals since libreduction shortcut
             #  cannot handle MultiIndex
-            and not self.agg_axis._has_complex_internals
+            and not isinstance(self.agg_axis, ABCMultiIndex)
         ):
 
             values = self.values

pandas/core/groupby/ops.py

@@ -15,6 +15,7 @@
 import pandas._libs.reduction as libreduction
 from pandas.errors import AbstractMethodError
 from pandas.util._decorators import cache_readonly
+from pandas.core.dtypes.generic import ABCMultiIndex
 
 from pandas.core.dtypes.common import (
     ensure_float64,
@@ -201,7 +202,7 @@ def apply(self, f, data, axis=0):
             and axis == 0
             # with MultiIndex, apply_frame_axis0 would raise InvalidApply
             # TODO: can we make this check prettier?
-            and not splitter._get_sorted_data().index._has_complex_internals
+            and not isinstance(splitter._get_sorted_data().index, ABCMultiIndex)
         ):
             try:
                 result_values, mutated = splitter.fast_apply(f, group_keys)
@@ -212,7 +213,7 @@ def apply(self, f, data, axis=0):
                     return group_keys, result_values, mutated
 
             except libreduction.InvalidApply as err:
-                # Cannot fast apply on MultiIndex (_has_complex_internals).
+                # Cannot fast apply on MultiIndex.
                 # This Exception is also raised if `f` triggers an exception
                 # but it is preferable to raise the exception in Python.
                 if "Let this error raise above us" not in str(err):
@@ -698,7 +699,7 @@ def _aggregate_series_fast(self, obj, func):
 
         # TODO: pre-empt this, also pre-empt get_result raising TypError if we pass a EA
         #   for EAs backed by ndarray we may have a performant workaround
-        if obj.index._has_complex_internals:
+        if isinstance(obj.index, ABCMultiIndex)
             raise TypeError("Incompatible index for Cython grouper")
 
         group_index, _, ngroups = self.group_info

pandas/core/indexes/base.py

@@ -4089,11 +4089,6 @@ def _assert_can_do_op(self, value):
             msg = "'value' must be a scalar, passed: {0}"
             raise TypeError(msg.format(type(value).__name__))
 
-    @property
-    def _has_complex_internals(self):
-        # to disable groupby tricks in MultiIndex
-        return False
-
     def _is_memory_usage_qualified(self):
         """
         Return a boolean if we need a qualified .info display.

pandas/core/indexes/multi.py

@@ -1394,11 +1394,6 @@ def values(self):
         self._tuples = lib.fast_zip(values)
         return self._tuples
 
-    @property
-    def _has_complex_internals(self):
-        # to disable groupby tricks
-        return True
-
     @cache_readonly
     def is_monotonic_increasing(self):
         """