Skip to content

Commit fbeb206

Browse files
author
Nico Cernek
committed
rename vars and add comment for clarity
1 parent 2dcd311 commit fbeb206

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

pandas/core/reshape/merge.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1292,17 +1292,19 @@ def _get_join_indexers(left_keys, right_keys, sort=False, how="inner", **kwargs)
12921292
# `count` is the num. of unique keys
12931293
# set(lkey) | set(rkey) == range(count)
12941294

1295+
# flip left and right keys if performing a right merge
1296+
# to preserve right merge row order (GH 27453)
12951297
if how == "right":
1296-
rkey, lkey, count = fkeys(rkey, lkey)
1298+
factorized_rkey, factorized_lkey, count = fkeys(rkey, lkey)
12971299
else:
1298-
lkey, rkey, count = fkeys(lkey, rkey)
1300+
factorized_lkey, factorized_rkey, count = fkeys(lkey, rkey)
12991301
# preserve left frame order if how == 'left' and sort == False
13001302
kwargs = copy.copy(kwargs)
13011303
if how == "left":
13021304
kwargs["sort"] = sort
13031305
join_func = _join_functions[how]
13041306

1305-
return join_func(lkey, rkey, count, **kwargs)
1307+
return join_func(factorized_lkey, factorized_rkey, count, **kwargs)
13061308

13071309

13081310
def _restore_dropped_levels_multijoin(

0 commit comments

Comments
 (0)