RE: [PATCH] Improvements to "Getting started" tutorial for GoogleCode-in task - Mailing list pgsql-hackers

From LAM JUN RONG
Subject RE: [PATCH] Improvements to "Getting started" tutorial for GoogleCode-in task
Date
Msg-id SG2PR01MB22990F1E2DA9FFEE98893DFAA0C80@SG2PR01MB2299.apcprd01.prod.exchangelabs.com
Whole thread Raw
In response to Re: [PATCH] Improvements to "Getting started" tutorial for GoogleCode-in task  (LAM JUN RONG <h1710074@nushigh.edu.sg>)
Responses Re: [PATCH] Improvements to "Getting started" tutorial for GoogleCode-in task
List pgsql-hackers

Hi,

 

I have made some changes based on Andreas’ suggestions.

 

> +    then one or more of the packages <productname>PostgreSQL</productname> requires is not installed.

> +    See <xref linkend="install-requirements"/> for the required packages.

>

> How about:

> then packages which are required to build

> <productname>PostgreSQL</productname> are missing.

> See <xref linkend="install-requirements"/> for a list of requirements.

 

This seems better than mine, I have included it.

 

>      If you are not sure whether <productname>PostgreSQL</productname>

> -    is already available or whether you can use it for your

> -    experimentation then you can install it yourself.  Doing so is not

> +    is already available for your experimentation,

> +    you can install it yourself.  Doing so is not

>      hard and it can be a good exercise.

>

> This change does not make much sense, given that you leave the

> second part of the sentence.

 

How’s this:

If you are not sure whether <productname>PostgreSQL</productname>

is already available for your experimentation,

you can install it yourself, which is not hard.

 

>      As is typical of client/server applications, the client and the

> -    server can be on different hosts.  In that case they communicate

> +    server can be on different machines or networks.  In that case they communicate

>      over a TCP/IP network connection.  You should keep this in mind,

> I think "hosts" is preferred here. The differentiation between machines

> and networks is not necessary.

>

>

>

> -    The path at your site might be different.  Contact your site

> +    The path at your site's server might be different.  Contact your site

>      administrator or check the installation instructions to

> Dunno if this change improves the wording, or the understanding.

> How about replacing "site" with "installation"?

 

For the 2 points above, I decided that the original documentation seems fine.

 

>      <productname>PostgreSQL</productname> allows you to create any

> -    number of databases at a given site.  Database names must have an

> -    alphabetic first character and are limited to 63 bytes in

> -    length.  A convenient choice is to create a database with the same

> -    name as your current user name.  Many tools assume that database

> -    name as the default, so it can save you some typing.  To create

> -    that database, simply type:

> +    number of databases at a given site.  Database names are limited to 63 bytes in

> +    length. Database names longer than 63 bytes will be truncated. A convenient

> +    choice is to create a database with the same name as your current user name.

> +    Many tools assume that database name as the default, so it

> +    can save you some typing. To create that database, simply type:

> The part about "truncate" is correct, maybe you can include NAMEDATALEN here as well.

> The part about the first character is correct - except you quote the name.

> Then any name is allowed. If you rewrite this part, maybe include this as well.

 

I’ve made some changes to include the part about NAMEDATALEN and quoting:

However, if you would like to create databases with

names that do not start with an alphabetic character, you will need to quote it like so: "1234".

Database names are limited to 63 bytes in length. Database names longer than 63 bytes will be

truncated. You can change this limit by modifying the NAMEDATALEN variable,

but that would require recompiling <productname>PostgreSQL</productname>.

 

 

A full diff is attached.

 

Thank you,

Jun Rong

From: LAM JUN RONG
Sent: Sunday, November 4, 2018 6:17 AM
Subject: Re: [PATCH] Improvements to "Getting started" tutorial for Google Code-in task

 

Thanks for your feedback.

I'll make some changes based on your suggestions and send a new diff.

Thanks!

From: Andreas 'ads' Scherbaum <ads@pgug.de>

Sent: Saturday, November 3, 2018 9:17:54 PM

To: LAM JUN RONG; pgsql-hackers@postgresql.org

Subject: Re: [PATCH] Improvements to "Getting started" tutorial for Google Code-in task

 

Hello,

On 02.11.18 14:07, LAM JUN RONG wrote:

 

I’m a student taking part in Google Code-in 2018. The task I am currently working on, https://codein.withgoogle.com/dashboard/task-instances/6406170207059968/, requires that I review and improve the “Getting Started” tutorial in the PostgreSQL docs, and submit a patch to this mailing list.

 

Thank you for picking up this task.

 

After reviewing the documentation, I found some parts to be slightly unclear. For example, in section 1.3 on creating databases, I found “no response” a bit unclear or ambiguous, so I replaced it with “exit without any error messages”.

 

After some experimentation, I found that a part of the documentation was incorrect. In Section 1.3, it was stated that “Database names must have an alphabetic first character”. However, I was able to create databases with names like “1234”, “$” or even “😀😀”. Hence, I decided to remove that sentence.

 

A diff of my changes is attached.

 

+    then one or more of the packages <productname>PostgreSQL</productname> requires is not installed.

+    See <xref linkend="install-requirements"/> for the required packages.

How about:

then packages which are required to build

<productname>PostgreSQL</productname> are missing.

See <xref linkend="install-requirements"/> for a list of requirements.



     If you are not sure whether <productname>PostgreSQL</productname>

-    is already available or whether you can use it for your

-    experimentation then you can install it yourself.  Doing so is not

+    is already available for your experimentation,

+    you can install it yourself.  Doing so is not

     hard and it can be a good exercise.

This change does not make much sense, given that you leave the

second part of the sentence.


     As is typical of client/server applications, the client and the

-    server can be on different hosts.  In that case they communicate

+    server can be on different machines or networks.  In that case they communicate

     over a TCP/IP network connection.  You should keep this in mind,

I think "hosts" is preferred here. The differentiation between machines

and networks is not necessary.



-    The path at your site might be different.  Contact your site

+    The path at your site's server might be different.  Contact your site

     administrator or check the installation instructions to

Dunno if this change improves the wording, or the understanding.

How about replacing "site" with "installation"?


     <productname>PostgreSQL</productname> allows you to create any

-    number of databases at a given site.  Database names must have an

-    alphabetic first character and are limited to 63 bytes in

-    length.  A convenient choice is to create a database with the same

-    name as your current user name.  Many tools assume that database

-    name as the default, so it can save you some typing.  To create

-    that database, simply type:

+    number of databases at a given site.  Database names are limited to 63 bytes in

+    length. Database names longer than 63 bytes will be truncated. A convenient

+    choice is to create a database with the same name as your current user name.

+    Many tools assume that database name as the default, so it

+    can save you some typing. To create that database, simply type:

The part about "truncate" is correct, maybe you can include NAMEDATALEN here as well.

The part about the first character is correct - except you quote the name.

Then any name is allowed. If you rewrite this part, maybe include this as well.

The other changes look good.

-- Andreas 'ads' Scherbaum German PostgreSQL User Group European PostgreSQL User Group - Board of Directors Volunteer Regional Contact, Germany - PostgreSQL Project

 

Attachment

pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: heap_sync seems rather oblivious to partitioned tables (wal_level=minimal)
Next
From: Steve Singer
Date:
Subject: Re: settings to control SSL/TLS protocol version