Re: "debug_invalidate_system_caches_always" is too long

2021-07-12 Thread Tom Lane
Noah Misch writes: > On Thu, Jul 08, 2021 at 04:34:55PM -0400, Tom Lane wrote: >> Robert Haas writes: >>> I like debug_discard_caches best. >> I can live with that. Anyone strongly against it? > I like it. Hearing no votes against, here's a proposed patch for that. (This is for HEAD; I expec

Re: "debug_invalidate_system_caches_always" is too long

2021-07-09 Thread Noah Misch
On Thu, Jul 08, 2021 at 04:34:55PM -0400, Tom Lane wrote: > Robert Haas writes: > > On Wed, Jul 7, 2021 at 11:17 AM Tom Lane wrote: > >> Fair point. What do you think of the alternative proposals > >> "debug_flush_caches", "debug_discard_caches", etc? > > > I like debug_discard_caches best. >

Re: "debug_invalidate_system_caches_always" is too long

2021-07-08 Thread Alvaro Herrera
On 2021-Jul-08, Tom Lane wrote: > Robert Haas writes: > > On Wed, Jul 7, 2021 at 11:17 AM Tom Lane wrote: > >> Fair point. What do you think of the alternative proposals > >> "debug_flush_caches", "debug_discard_caches", etc? > > > I like debug_discard_caches best. > > I can live with that.

Re: "debug_invalidate_system_caches_always" is too long

2021-07-08 Thread Tom Lane
Robert Haas writes: > On Wed, Jul 7, 2021 at 11:17 AM Tom Lane wrote: >> Fair point. What do you think of the alternative proposals >> "debug_flush_caches", "debug_discard_caches", etc? > I like debug_discard_caches best. I can live with that. Anyone strongly against it?

Re: "debug_invalidate_system_caches_always" is too long

2021-07-08 Thread Robert Haas
On Wed, Jul 7, 2021 at 11:17 AM Tom Lane wrote: > Peter Eisentraut writes: > > The clobbering doesn't actually happen unless you turn on > > CLOBBER_FREED_MEMORY, so it would be good to keep that separate. > > Fair point. What do you think of the alternative proposals > "debug_flush_caches", "de

Re: "debug_invalidate_system_caches_always" is too long

2021-07-07 Thread Tom Lane
Peter Eisentraut writes: > The clobbering doesn't actually happen unless you turn on > CLOBBER_FREED_MEMORY, so it would be good to keep that separate. Fair point. What do you think of the alternative proposals "debug_flush_caches", "debug_discard_caches", etc? regards,

Re: "debug_invalidate_system_caches_always" is too long

2021-07-07 Thread Peter Eisentraut
On 04.07.21 22:27, Tom Lane wrote: I do agree with the "debug_" prefix given that it's now visible to users. However, it doesn't seem that hard to save some space in the rest of the name. The word "system" is adding nothing of value, and the word "always" seems rather confusing --- if it does s

Re: "debug_invalidate_system_caches_always" is too long

2021-07-06 Thread Tom Lane
Andrew Dunstan writes: > On 7/5/21 11:46 PM, Bharath Rupireddy wrote: >> On Tue, Jul 6, 2021 at 12:43 AM Tom Lane wrote: >>> I like "debug_flush_caches" --- it's short and accurate. >> Do we always flush the cache entries into the disk? Sometimes we just >> invalidate the cache entries in the re

Re: "debug_invalidate_system_caches_always" is too long

2021-07-06 Thread Andrew Dunstan
On 7/5/21 11:46 PM, Bharath Rupireddy wrote: > On Tue, Jul 6, 2021 at 12:43 AM Tom Lane wrote: >> Noah Misch writes: >>> On Sun, Jul 04, 2021 at 04:27:13PM -0400, Tom Lane wrote: However, I think we should also give serious consideration to "debug_clobber_cache" or "debug_clobber_cach

Re: "debug_invalidate_system_caches_always" is too long

2021-07-05 Thread Bharath Rupireddy
On Tue, Jul 6, 2021 at 12:43 AM Tom Lane wrote: > > Noah Misch writes: > > On Sun, Jul 04, 2021 at 04:27:13PM -0400, Tom Lane wrote: > >> However, I think we should also give serious consideration to > >> "debug_clobber_cache" or "debug_clobber_cache_always" for continuity > >> with past practice

Re: "debug_invalidate_system_caches_always" is too long

2021-07-05 Thread Tom Lane
Noah Misch writes: > On Sun, Jul 04, 2021 at 04:27:13PM -0400, Tom Lane wrote: >> However, I think we should also give serious consideration to >> "debug_clobber_cache" or "debug_clobber_cache_always" for continuity >> with past practice (though it still feels like "always" is a good >> word to lo

Re: "debug_invalidate_system_caches_always" is too long

2021-07-05 Thread Andrew Dunstan
On 7/4/21 4:27 PM, Tom Lane wrote: > As I've been poking around in this area, I find myself growing > increasingly annoyed at the new GUC name > "debug_invalidate_system_caches_always". It is too d*mn long. > It's a serious pain to type in any context where you don't have > autocomplete to help

Re: "debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Bharath Rupireddy
On Mon, Jul 5, 2021 at 1:57 AM Tom Lane wrote: > > As I've been poking around in this area, I find myself growing > increasingly annoyed at the new GUC name > "debug_invalidate_system_caches_always". It is too d*mn long. > It's a serious pain to type in any context where you don't have > autocomp

Re: "debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Amul Sul
On Mon, Jul 5, 2021 at 1:57 AM Tom Lane wrote: > > As I've been poking around in this area, I find myself growing > increasingly annoyed at the new GUC name > "debug_invalidate_system_caches_always". It is too d*mn long. > It's a serious pain to type in any context where you don't have > autocomp

Re: "debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Kyotaro Horiguchi
At Sun, 4 Jul 2021 14:12:34 -0700, Noah Misch wrote in > > However, I think we should also give serious consideration to > > "debug_clobber_cache" or "debug_clobber_cache_always" for continuity > > with past practice (though it still feels like "always" is a good > > word to lose now). "debug_cl

Re: "debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Justin Pryzby
On Sun, Jul 04, 2021 at 04:27:13PM -0400, Tom Lane wrote: > and the word "always" seems rather confusing --- if it does > something "always", why is there more than one level? So a simple > proposal is to rename it to "debug_invalidate_caches". +1 to remove "always" -- Justin

Re: "debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Noah Misch
On Sun, Jul 04, 2021 at 04:27:13PM -0400, Tom Lane wrote: > As I've been poking around in this area, I find myself growing > increasingly annoyed at the new GUC name > "debug_invalidate_system_caches_always". It is too d*mn long. > It's a serious pain to type in any context where you don't have >

"debug_invalidate_system_caches_always" is too long

2021-07-04 Thread Tom Lane
As I've been poking around in this area, I find myself growing increasingly annoyed at the new GUC name "debug_invalidate_system_caches_always". It is too d*mn long. It's a serious pain to type in any context where you don't have autocomplete to help you. I've kept referring to this type of testi