Re-enable abbreviated keys on Windows.
authorRobert Haas <[email protected]>
Mon, 26 Jan 2015 19:28:14 +0000 (14:28 -0500)
committerRobert Haas <[email protected]>
Mon, 26 Jan 2015 19:28:14 +0000 (14:28 -0500)
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

index f48ce4901ca154be8b00e9cfe0e6ba979a59e71c..3edd283212cc596da052b6623acdcf20b11085b1 100644 (file)
@@ -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