From 73dd71628562e6075e9601e4860429aa41e2c0dc Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Wed, 1 Mar 2000 17:14:09 +0000 Subject: [PATCH] Small performance improvement in comparetup_heap. --- src/backend/utils/sort/tuplesort.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/backend/utils/sort/tuplesort.c b/src/backend/utils/sort/tuplesort.c index be80b246a2..7998372d4b 100644 --- a/src/backend/utils/sort/tuplesort.c +++ b/src/backend/utils/sort/tuplesort.c @@ -78,7 +78,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/utils/sort/tuplesort.c,v 1.6 2000/02/18 06:32:30 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/utils/sort/tuplesort.c,v 1.7 2000/03/01 17:14:09 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -1632,20 +1632,16 @@ comparetup_heap(Tuplesortstate *state, const void *a, const void *b) for (nkey = 0; nkey < state->nKeys; nkey++) { ScanKey scanKey = state->scanKeys + nkey; + AttrNumber attno = scanKey->sk_attno; Datum lattr, rattr; bool isnull1, isnull2; int result; - lattr = heap_getattr(ltup, - scanKey->sk_attno, - tupDesc, - &isnull1); - rattr = heap_getattr(rtup, - scanKey->sk_attno, - tupDesc, - &isnull2); + lattr = heap_getattr(ltup, attno, tupDesc, &isnull1); + rattr = heap_getattr(rtup, attno, tupDesc, &isnull2); + if (isnull1) { if (!isnull2)