Re: Remove unnecessary "lmgr.h" in stat_utils.c - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Remove unnecessary "lmgr.h" in stat_utils.c
Date
Msg-id aBqk6Tsex4WP2ho1@paquier.xyz
Whole thread Raw
In response to Remove unnecessary "lmgr.h" in stat_utils.c  (Ilia Evdokimov <ilya.evdokimov@tantorlabs.com>)
Responses Re: Remove unnecessary "lmgr.h" in stat_utils.c
List pgsql-hackers
On Mon, May 05, 2025 at 04:33:20PM +0300, Ilia Evdokimov wrote:
> While reviewing the import/export statistics, I noticed that relation
> locking are handled via relation_open() and relation_close() in
> stats_lock_check_privileges(), and no calls to other lock-manager routines
> are actually used there. As a result, the inclusion of the lock-manager
> header
>
> #include "storage/lmgr.h"
>
> is not needed. I have attached a small patch which simply removes that
> include.

True that we try to be clean when it comes to that.

I suspect that this is an artifact from one of the original versions
of the patch compared to what has been committed when this feature has
been discussed.  Corey?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: minor fix related to Auxiliary processes and IO workers
Next
From: Mark Dilger
Date:
Subject: Re: Adding skip scan (including MDAM style range skip scan) to nbtree