Remove assertion in pgstat_count_io_op()
authorMichael Paquier <[email protected]>
Tue, 14 Jan 2025 03:19:51 +0000 (12:19 +0900)
committerMichael Paquier <[email protected]>
Tue, 14 Jan 2025 03:19:51 +0000 (12:19 +0900)
An equivalent check is done with pgstat_is_ioop_tracked_in_bytes(), so
there is no need for this extra one.  Small cleanup that should have
been included in f92c854cf406.

Author: Nazir Bilal Yavuz
Reviewed-by: Bertrand Drouvot
Discussion: https://postgr.es/m/CAN55FZ0oqxBaaHAEsj=xFqkzE3n5P=3RA1V_igXwL-RV7QRzyw@mail.gmail.com

src/backend/utils/activity/pgstat_io.c

index 78b01310d8b4527412d27876977b9ad0045382c2..e03b021af3d2f91a37b756c44c728adebe0c2bf1 100644 (file)
@@ -78,7 +78,6 @@ pgstat_count_io_op(IOObject io_object, IOContext io_context, IOOp io_op,
 {
        Assert((unsigned int) io_object < IOOBJECT_NUM_TYPES);
        Assert((unsigned int) io_context < IOCONTEXT_NUM_TYPES);
-       Assert((unsigned int) io_op < IOOP_NUM_TYPES);
        Assert(pgstat_is_ioop_tracked_in_bytes(io_op) || bytes == 0);
        Assert(pgstat_tracks_io_op(MyBackendType, io_object, io_context, io_op));