Incorrect grammar

Started by Etsuro Fujitaalmost 8 years ago3 messages

fujita.etsuro@lab.ntt.co.jp

almost 8 years ago

1 attachment(s)

Hi,

While reviewing the
lazy-initialization-of-partition-info-for-tuple-routing patch, I ran
into a grammar mistake in a comment in ExecSetupChildParentMapForLeaf.
Attached is a patch for fixing that.

Best regards,
Etsuro Fujita

Attachments:

fix-incorrect-grammar.patchtext/x-diff; name=fix-incorrect-grammar.patchDownload

diff --git a/src/backend/executor/execPartition.c b/src/backend/executor/execPartition.c
index 106a96d..ba6b52c 100644
--- a/src/backend/executor/execPartition.c
+++ b/src/backend/executor/execPartition.c
@@ -360,7 +360,7 @@ ExecSetupChildParentMapForLeaf(PartitionTupleRouting *proute)
 	Assert(proute != NULL);
 
 	/*
-	 * These array elements gets filled up with maps on an on-demand basis.
+	 * These array elements get filled up with maps on an on-demand basis.
 	 * Initially just set all of them to NULL.
 	 */
 	proute->child_parent_tupconv_maps =

Robert Haas

robertmhaas@gmail.com

almost 8 years ago

In reply to: Etsuro Fujita (#1)

Re: Incorrect grammar

On Tue, Feb 6, 2018 at 5:22 AM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:

While reviewing the
lazy-initialization-of-partition-info-for-tuple-routing patch, I ran
into a grammar mistake in a comment in ExecSetupChildParentMapForLeaf.
Attached is a patch for fixing that.

Committed.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Etsuro Fujita

fujita.etsuro@lab.ntt.co.jp

almost 8 years ago

In reply to: Robert Haas (#2)

Re: Incorrect grammar

(2018/02/07 5:51), Robert Haas wrote:

On Tue, Feb 6, 2018 at 5:22 AM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:

While reviewing the
lazy-initialization-of-partition-info-for-tuple-routing patch, I ran
into a grammar mistake in a comment in ExecSetupChildParentMapForLeaf.
Attached is a patch for fixing that.

Committed.

Thank you.

Best regards,
Etsuro Fujita