gpdb icon indicating copy to clipboard operation
gpdb copied to clipboard

Fix PARTITION typo.

Open avamingli opened this issue 1 year ago • 8 comments

Authored-by: Zhang Mingli [email protected]

Here are some reminders before you submit the pull request

  • [ ] Add tests for the change
  • [ ] Document changes
  • [ ] Communicate in the mailing list if needed
  • [ ] Pass make installcheck
  • [ ] Review a PR in return to support the community

avamingli avatar Apr 04 '24 01:04 avamingli

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL. 😂 Would you like to fix them?

grep -riE '(parttion|partion|partiton|parition|parittion)'

higuoxing avatar Apr 04 '24 02:04 higuoxing

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL. 😂 Would you like to fix them?

grep -riE '(parttion|partion|partiton|parition|parittion)'

Good catch.Will do.

avamingli avatar Apr 04 '24 02:04 avamingli

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL.

I exec your command to search in Postgres:

grep -riE '(parttion|partion|partiton|parition|parittion)'
src/bin/pg_dump/po/ka.po:msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"
src/backend/po/tr.po:#~ msgstr "silinecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"
src/backend/po/tr.po:#~ msgstr "güncellenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"
src/backend/po/de.po:msgstr "Der neue Modulus %d ist nicht durch %d, den Modulus der bestehenden Parition »%s«, teilbar."
src/backend/po/sv.po:msgstr "villkoret %s på tabell \"%s\" saknar kolumn \"%s\" som är en del av partioneringsnyckeln."
src/backend/po/sv.po:msgstr "ALTER TABLE / ADD CONSTRAINT USING INDEX stöds inte på partionerade tabeller"
src/backend/po/sv.po:msgstr "kan inte använda list-partioneringsstrategi med mer än en kolumn"
src/backend/po/sv.po:msgstr "Ny modulon %d är inte delbar med %d som är modulo för den existerande paritionen \"%s\"."
src/backend/po/sv.po:msgstr "antalet partitioneringskolumner (%d) stämmer inte med antalet partioneringsnycklas som angivits (%d)"

They are all in *.po, I think in all probability there are no problems in them since different languages have their own spellings. I have a little experience that some language's spelling have more or less letter with English words, but not 100% sure for them all.

So it may be not a problem for Postgres, I will have a check on Greenplum later.

avamingli avatar Apr 04 '24 02:04 avamingli

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL.

I exec your command to search in Postgres:

grep -riE '(parttion|partion|partiton|parition|parittion)'
src/bin/pg_dump/po/ka.po:msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"

I think it's a typo, it should be "pg_partitioned_table".

#: pg_dump.c:6919
#, c-format
msgid "failed sanity check, table OID %u appearing in pg_partitioned_table not found"
msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"

src/backend/po/tr.po:#~ msgstr "silinecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış" src/backend/po/tr.po:#~ msgstr "güncellenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"

Same file in line 763 is using "partition"

#: access/heap/heapam_handler.c:405
#, c-format
msgid "tuple to be locked was already moved to another partition due to concurrent update"
msgstr "kilitlenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partition) taşınmış"

src/backend/po/de.po:msgstr "Der neue Modulus %d ist nicht durch %d, den Modulus der bestehenden Parition »%s«, teilbar."

Same file in line 18949 is using "Partition"

#: partitioning/partbounds.c:2974 partitioning/partbounds.c:3015
#, c-format
msgid "The new modulus %d is not a factor of %d, the modulus of existing partition \"%s\"."
msgstr "Der neue Modulus %d ist kein Faktor von %d, dem Modulus der bestehenden Partition »%s«."

src/backend/po/sv.po:msgstr "villkoret %s på tabell "%s" saknar kolumn "%s" som är en del av partioneringsnyckeln."

Same file in line 19055 is using "partitioneringsnyckeln"

#: partitioning/partbounds.c:4864
#, c-format
msgid "column %d of the partition key has type %s, but supplied value is of type %s"
msgstr "kolumn %d i partitioneringsnyckeln har typ %s men använt värde har typ %s"

src/backend/po/sv.po:msgstr "ALTER TABLE / ADD CONSTRAINT USING INDEX stöds inte på partionerade tabeller"

Same file in line 5584 is using "partitionerade tabeller"

#: catalog/pg_class.c:45
#, c-format
msgid "This operation is not supported for partitioned tables."
msgstr "Denna operation stöds inte på partitionerade tabeller."

src/backend/po/sv.po:msgstr "kan inte använda list-partioneringsstrategi med mer än en kolumn"

Same file in line 14240 is using "partitioneringsstrategi"

#: gram.y:18742
#, c-format
msgid "unrecognized partitioning strategy \"%s\""
msgstr "okänd partitioneringsstrategi \"%s\""

src/backend/po/sv.po:msgstr "Ny modulon %d är inte delbar med %d som är modulo för den existerande paritionen "%s"." src/backend/po/sv.po:msgstr "antalet partitioneringskolumner (%d) stämmer inte med antalet partioneringsnycklas som angivits (%d)"

Cannot find a similar word but it looks like a typo.

higuoxing avatar Apr 04 '24 02:04 higuoxing

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL.

I exec your command to search in Postgres:

grep -riE '(parttion|partion|partiton|parition|parittion)'

src/bin/pg_dump/po/ka.po:msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"

I think it's a typo, it should be "pg_partitioned_table".


#: pg_dump.c:6919

#, c-format

msgid "failed sanity check, table OID %u appearing in pg_partitioned_table not found"

msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"


src/backend/po/tr.po:#~ msgstr "silinecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"

src/backend/po/tr.po:#~ msgstr "güncellenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"

Same file in line 763 is using "partition"


#: access/heap/heapam_handler.c:405

#, c-format

msgid "tuple to be locked was already moved to another partition due to concurrent update"

msgstr "kilitlenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partition) taşınmış"


src/backend/po/de.po:msgstr "Der neue Modulus %d ist nicht durch %d, den Modulus der bestehenden Parition »%s«, teilbar."

Same file in line 18949 is using "Partition"


#: partitioning/partbounds.c:2974 partitioning/partbounds.c:3015

#, c-format

msgid "The new modulus %d is not a factor of %d, the modulus of existing partition \"%s\"."

msgstr "Der neue Modulus %d ist kein Faktor von %d, dem Modulus der bestehenden Partition »%s«."


src/backend/po/sv.po:msgstr "villkoret %s på tabell "%s" saknar kolumn "%s" som är en del av partioneringsnyckeln."

Same file in line 19055 is using "partitioneringsnyckeln"


#: partitioning/partbounds.c:4864

#, c-format

msgid "column %d of the partition key has type %s, but supplied value is of type %s"

msgstr "kolumn %d i partitioneringsnyckeln har typ %s men använt värde har typ %s"


src/backend/po/sv.po:msgstr "ALTER TABLE / ADD CONSTRAINT USING INDEX stöds inte på partionerade tabeller"

Same file in line 5584 is using "partitionerade tabeller"


#: catalog/pg_class.c:45

#, c-format

msgid "This operation is not supported for partitioned tables."

msgstr "Denna operation stöds inte på partitionerade tabeller."


src/backend/po/sv.po:msgstr "kan inte använda list-partioneringsstrategi med mer än en kolumn"

Same file in line 14240 is using "partitioneringsstrategi"


#: gram.y:18742

#, c-format

msgid "unrecognized partitioning strategy \"%s\""

msgstr "okänd partitioneringsstrategi \"%s\""


src/backend/po/sv.po:msgstr "Ny modulon %d är inte delbar med %d som är modulo för den existerande paritionen "%s"."

src/backend/po/sv.po:msgstr "antalet partitioneringskolumner (%d) stämmer inte med antalet partioneringsnycklas som angivits (%d)"

Cannot find a similar word but it looks like a typo.

Sounds reasonable, maybe we should submit a patch for Postgres.

avamingli avatar Apr 04 '24 03:04 avamingli

Sounds reasonable, maybe we should submit a patch for Postgres.

Great! Please go ahead!

higuoxing avatar Apr 04 '24 03:04 higuoxing

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL.

I exec your command to search in Postgres:

grep -riE '(parttion|partion|partiton|parition|parittion)'
src/bin/pg_dump/po/ka.po:msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"

I think it's a typo, it should be "pg_partitioned_table".

#: pg_dump.c:6919
#, c-format
msgid "failed sanity check, table OID %u appearing in pg_partitioned_table not found"
msgstr "სისწორის შემოწმების შეცდომა. pg_parttioned_table-ში მოხსენიებული ცხრილი OID-ით %u ვერ ვიპოვე"

src/backend/po/tr.po:#~ msgstr "silinecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış" src/backend/po/tr.po:#~ msgstr "güncellenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partiton) taşınmış"

Same file in line 763 is using "partition"

#: access/heap/heapam_handler.c:405
#, c-format
msgid "tuple to be locked was already moved to another partition due to concurrent update"
msgstr "kilitlenecek satır eş zamanlı bir güncelleme dolayısıyla zaten başka bir bölüme (partition) taşınmış"

src/backend/po/de.po:msgstr "Der neue Modulus %d ist nicht durch %d, den Modulus der bestehenden Parition »%s«, teilbar."

Same file in line 18949 is using "Partition"

#: partitioning/partbounds.c:2974 partitioning/partbounds.c:3015
#, c-format
msgid "The new modulus %d is not a factor of %d, the modulus of existing partition \"%s\"."
msgstr "Der neue Modulus %d ist kein Faktor von %d, dem Modulus der bestehenden Partition »%s«."

src/backend/po/sv.po:msgstr "villkoret %s på tabell "%s" saknar kolumn "%s" som är en del av partioneringsnyckeln."

Same file in line 19055 is using "partitioneringsnyckeln"

#: partitioning/partbounds.c:4864
#, c-format
msgid "column %d of the partition key has type %s, but supplied value is of type %s"
msgstr "kolumn %d i partitioneringsnyckeln har typ %s men använt värde har typ %s"

src/backend/po/sv.po:msgstr "ALTER TABLE / ADD CONSTRAINT USING INDEX stöds inte på partionerade tabeller"

Same file in line 5584 is using "partitionerade tabeller"

#: catalog/pg_class.c:45
#, c-format
msgid "This operation is not supported for partitioned tables."
msgstr "Denna operation stöds inte på partitionerade tabeller."

src/backend/po/sv.po:msgstr "kan inte använda list-partioneringsstrategi med mer än en kolumn"

Same file in line 14240 is using "partitioneringsstrategi"

#: gram.y:18742
#, c-format
msgid "unrecognized partitioning strategy \"%s\""
msgstr "okänd partitioneringsstrategi \"%s\""

src/backend/po/sv.po:msgstr "Ny modulon %d är inte delbar med %d som är modulo för den existerande paritionen "%s"." src/backend/po/sv.po:msgstr "antalet partitioneringskolumner (%d) stämmer inte med antalet partioneringsnycklas som angivits (%d)"

Cannot find a similar word but it looks like a typo.

amazing work

yihong0618 avatar Apr 04 '24 03:04 yihong0618

There're quite some typos relevent to partitioned tables both in Greenplum and PostgreSQL. 😂 Would you like to fix them?

grep -riE '(parttion|partion|partiton|parition|parittion)'

Fix all those in GPDB.

avamingli avatar Apr 22 '24 09:04 avamingli