I took a look at your patch. Couple of notes from me.
const char * GetEventIdentifier(uint16 eventId) { const char *res; switch (eventId) { case EVENT_ARCHIVER_MAIN: res = "ArchiverMain"; break; ... long long list of events ... case EVENT_WAL_SENDER_WRITE_DATA: res = "WalSenderWriteData"; break; default: res = "???"; } return res; }
Would it be better to use an array here?
typedef enum EventIdentifier {
EventIdentifier seems too general name for me, isn't it? Could we name it WaitEventIdentifier? Or WaitEventId for shortcut?
I'm also not sure about handling of secure_read() and secure_write() functions.
In the current patch we're tracking latch event wait inside them. But we setup latch only for blocking sockets and can do it multiple times in loop.
Would it be better to make separate wait events NETWORK_READ and NETWORK_WRITE and setup them for the whole time spent in secure_read() and secure_write()?
