On Mon, Jan 15, 2024 at 10:49:56AM +0100, Alvaro Herrera wrote: > the new function pqPipelineSyncInternal is not a wrapper for these other > two functions -- the opposite is true actually. We tend to use the term > "workhorse" or "internal workhorse" for this kind of thing.
Indeed, makes sense. > In the docs, after this patch we have > > - PQpipelineSync > - PQsendFlushRequest > - PQsendPipelineSync > > Wouldn't it make more sense to add the new function in the middle of the > two existing ones instead? Ordering PQsendPipelineSync just after PQpipelineSync is OK by me. I've applied the patch with all these modifications to move on with the subject. > Looking again at the largish comment that's now atop > pqPipelineSyncInternal(), I think most of it should be removed -- these > things should be explained in the SGML docs, and I think they are, in > the "Using Pipeline Mode" section. We can just have the lines this > patch is adding. Hmm. The first two sentences about being able to submit more commands to the pipeline are documented in the subsection "Issuing Queries". The third sentence is implied in the second paragraph of this subsection. The 4th paragraph of the comment where sync commands cannot be issued until all the results from the pipeline have been consumed is mentioned in the first paragraph in "Using Pipeline Mode". So you are right that this could be entirely removed. How about the attached to remove all that, then? -- Michael
diff --git a/src/interfaces/libpq/fe-exec.c b/src/interfaces/libpq/fe-exec.c index 52d41658c1..152b100624 100644 --- a/src/interfaces/libpq/fe-exec.c +++ b/src/interfaces/libpq/fe-exec.c @@ -3245,23 +3245,6 @@ PQsendPipelineSync(PGconn *conn) /* * Workhorse function for PQpipelineSync and PQsendPipelineSync. * - * It's legal to start submitting more commands in the pipeline immediately, - * without waiting for the results of the current pipeline. There's no need to - * end pipeline mode and start it again. - * - * If a command in a pipeline fails, every subsequent command up to and - * including the result to the Sync message sent by pqPipelineSyncInternal - * gets set to PGRES_PIPELINE_ABORTED state. If the whole pipeline is - * processed without error, a PGresult with PGRES_PIPELINE_SYNC is produced. - * - * Queries can already have been sent before pqPipelineSyncInternal is called, - * but pqPipelineSyncInternal needs to be called before retrieving command - * results. - * - * The connection will remain in pipeline mode and unavailable for new - * synchronous command execution functions until all results from the pipeline - * are processed by the client. - * * immediate_flush controls if the flush happens immediately after sending the * Sync message or not. */
signature.asc
Description: PGP signature
