On Tue, Mar 21, 2017 at 3:16 PM, Seki, Eiji <seki.e...@jp.fujitsu.com>
> Thank you for you review.
> I reflected your comment and attach the updated patch.
Thanks for the updated patch.
+/* Use these flags in GetOldestXmin as "flags" */
How about some thing like the following.
/* Use the following flags as an input "flags" to GetOldestXmin function */
+/* Ignore vacuum backends (Note: this also ignores analyze with vacuum
+#define PROCARRAY_FLAGS_VACUUM PROCARRAY_FLAGS_DEFAULT |
+/* Ignore analyze backends (Note: this also ignores vacuum with analyze
+#define PROCARRAY_FLAGS_ANALYZE PROCARRAY_FLAGS_DEFAULT |
Whenever the above flags are passed to the GetOldestXmin() function,
it just verifies whether any one of the flags are set in the backend flags
or not. And also actually the PROC_IN_ANALYZE flag will set when
analyze operation is started and reset at the end. I feel, it is not
required to mention the Note section.
+/* Ignore vacuum backends and analyze ones */
How about changing it as "Ignore both vacuum and analyze backends".