From 168a809d4b8adb7cfd6c0f14116493c3f7bc24b7 Mon Sep 17 00:00:00 2001 From: Robert Haas Date: Mon, 26 Jan 2015 14:28:14 -0500 Subject: [PATCH] Re-enable abbreviated keys on Windows. Commit 1be4eb1b2d436d1375899c74e4c74486890d8777 disabled this, but I think the real problem here was fixed by commit b181a91981203f6ec9403115a2917bd3f9473707 and commit d060e07fa919e0eb681e2fa2cfbe63d6c40eb2cf. So let's try re-enabling it now and see what happens. --- src/backend/utils/adt/varlena.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/backend/utils/adt/varlena.c b/src/backend/utils/adt/varlena.c index f48ce4901ca..3edd283212c 100644 --- a/src/backend/utils/adt/varlena.c +++ b/src/backend/utils/adt/varlena.c @@ -1817,13 +1817,7 @@ btsortsupport_worker(SortSupport ssup, Oid collid) * keys optimization may win, and if it doesn't, the "abort abbreviation" * code may rescue us. So, for now, we don't disable this anywhere on the * basis of performance. - * - * On Windows, however, strxfrm() seems to be unreliable on some machines, - * so we categorically disable this optimization there. */ -#ifdef WIN32 - abbreviate = false; -#endif /* * If we're using abbreviated keys, or if we're using a locale-aware