Skip to content

Commit

Permalink
use 'should_keep_index' name
Browse files Browse the repository at this point in the history
Signed-off-by: Anatoly Myachev <[email protected]>
  • Loading branch information
anmyachev committed Nov 6, 2023
1 parent f439dc8 commit e19125f
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions modin/core/storage_formats/pandas/query_compiler.py
Original file line number Diff line number Diff line change
Expand Up @@ -499,7 +499,7 @@ def merge(self, right, **kwargs):

kwargs["sort"] = False

def whether_keep_index(left, right):
def should_keep_index(left, right):
keep_index = False
if left_on is not None and right_on is not None:
keep_index = any(
Expand All @@ -522,7 +522,7 @@ def map_func(
if kwargs["how"] == "left":
partition_idx = service_kwargs["partition_idx"]
if len(axis_lengths):
if not whether_keep_index(left, right):
if not should_keep_index(left, right):
# Doesn't work for "inner" case, since the partition sizes of the
# left dataframe may change
start = sum(axis_lengths[:partition_idx])
Expand Down Expand Up @@ -598,7 +598,7 @@ def map_func(
# materialized quite often compared to the indexes.
keep_index = False
if self._modin_frame.has_materialized_index:
keep_index = whether_keep_index(self, right_pandas)
keep_index = should_keep_index(self, right_pandas)
else:
# Have to trigger columns materialization. Hope they're already available at this point.
if left_on is not None and right_on is not None:
Expand Down

0 comments on commit e19125f

Please sign in to comment.