]> granicus.if.org Git - postgresql/commitdiff
pg_test_fsync: Update output format
authorPeter Eisentraut <peter_e@gmx.net>
Mon, 20 Oct 2014 19:34:59 +0000 (15:34 -0400)
committerPeter Eisentraut <peter_e@gmx.net>
Mon, 20 Oct 2014 19:36:51 +0000 (15:36 -0400)
Apparently, computers are now a bit faster than when this was first
added, so we need to make room for a digit or two in the ops/sec format.

While we're at it, adjust some of the other output for a more consistent
line length.

contrib/pg_test_fsync/pg_test_fsync.c

index 842295ae3d97681e24cf903f99be767b9620a7ca..3352f046dcad70a329aac52067bfbec66abb036a 100644 (file)
@@ -23,9 +23,9 @@
 
 #define XLOG_BLCKSZ_K  (XLOG_BLCKSZ / 1024)
 
-#define LABEL_FORMAT           "        %-32s"
+#define LABEL_FORMAT           "        %-30s"
 #define NA_FORMAT                      "%20s"
-#define OPS_FORMAT                     "%11.3f ops/sec  %6.0f usecs/op"
+#define OPS_FORMAT                     "%13.3f ops/sec  %6.0f usecs/op"
 #define USECS_SEC                      1000000
 
 /* These are macros to avoid timing the function call overhead. */
@@ -242,8 +242,7 @@ test_sync(int writes_per_op)
                printf("\nCompare file sync methods using one %dkB write:\n", XLOG_BLCKSZ_K);
        else
                printf("\nCompare file sync methods using two %dkB writes:\n", XLOG_BLCKSZ_K);
-       printf("(in wal_sync_method preference order, except fdatasync\n");
-       printf("is Linux's default)\n");
+       printf("(in wal_sync_method preference order, except fdatasync is Linux's default)\n");
 
        /*
         * Test open_datasync if available
@@ -398,8 +397,8 @@ static void
 test_open_syncs(void)
 {
        printf("\nCompare open_sync with different write sizes:\n");
-       printf("(This is designed to compare the cost of writing 16kB\n");
-       printf("in different write open_sync sizes.)\n");
+       printf("(This is designed to compare the cost of writing 16kB in different write\n"
+                  "open_sync sizes.)\n");
 
        test_open_sync(" 1 * 16kB open_sync write", 16);
        test_open_sync(" 2 *  8kB open_sync writes", 8);
@@ -459,8 +458,8 @@ test_file_descriptor_sync(void)
         * on platforms which support it.
         */
        printf("\nTest if fsync on non-write file descriptor is honored:\n");
-       printf("(If the times are similar, fsync() can sync data written\n");
-       printf("on a different descriptor.)\n");
+       printf("(If the times are similar, fsync() can sync data written on a different\n"
+                  "descriptor.)\n");
 
        /*
         * first write, fsync and close, which is the normal behavior without
@@ -524,7 +523,7 @@ test_non_sync(void)
        /*
         * Test a simple write without fsync
         */
-       printf("\nNon-Sync'ed %dkB writes:\n", XLOG_BLCKSZ_K);
+       printf("\nNon-sync'ed %dkB writes:\n", XLOG_BLCKSZ_K);
        printf(LABEL_FORMAT, "write");
        fflush(stdout);