part1: Simplify BaseIndex to an abstract class

[skirui-source]

This PR is in response to @vyasr comment, as partial fix for PR #9593 :

BaseIndex should be reduced as closely as possible to an abstract class. While there are a subset of APIs that truly make sense for all types of index objects, in almost all cases the optimal implementation for RangeIndex (and MultiIndex, for that matter) is very different from the implementation for GenericIndex. In addition, this change reduces cognitive load for developers by simplifying the inheritance hierarchy

[codecov]

Codecov Report

Base: 87.40% // Head: 87.49% // Increases project coverage by +0.08% 🎉

Coverage data is based on head (33c54c2) compared to base (f72c4ce).
Patch coverage: 84.48% of modified lines in pull request are covered.

❗ Current head 33c54c2 differs from pull request most recent head 407a2cc. Consider uploading reports for the commit 407a2cc to get more accurate results

Additional details and impacted files

@@               Coverage Diff                @@
##           branch-22.12   #10389      +/-   ##
================================================
+ Coverage         87.40%   87.49%   +0.08%     
================================================
  Files               133      133              
  Lines             21833    21864      +31     
================================================
+ Hits              19084    19129      +45     
+ Misses             2749     2735      -14     

Impacted Files	Coverage Δ
python/cudf/cudf/core/udf/__init__.py	50.00% <ø> (ø)
python/cudf/cudf/utils/ioutils.py	79.47% <ø> (ø)
...thon/dask_cudf/dask_cudf/tests/test_distributed.py	18.86% <ø> (+4.94%)	⬆️
python/cudf/cudf/core/_base_index.py	82.20% <43.75%> (-3.35%)	⬇️
python/strings_udf/strings_udf/__init__.py	86.27% <76.00%> (-10.61%)	⬇️
python/cudf/cudf/core/index.py	92.91% <95.08%> (+0.27%)	⬆️
python/cudf/cudf/__init__.py	90.69% <100.00%> (ø)
python/cudf/cudf/core/column/categorical.py	89.34% <100.00%> (ø)
python/cudf/cudf/core/scalar.py	90.52% <100.00%> (+1.25%)	⬆️
python/dask_cudf/dask_cudf/sorting.py	93.29% <100.00%> (+2.11%)	⬆️
... and 5 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[github-actions]

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

[vyasr]

I have two very minor comments that I would have been fine punting on, but there's a small bug in any that needs to get fixed so let's just address all the comments before merging. At this point I would retarget to 22.12 since we're in code freeze, but we should be able to merge this very soon.

[vyasr]

LGTM! Thanks for all the iterations.

[skirui-source]

@gpucibot rerun tests

[skirui-source]

@vyasr @mroeschke , need your eyes on this pr, the codecov/patch check keeps failing despite having added said tests cases.. is this a block to merging?

[vyasr]

@gpucibot merge