Re: Wrote a connect-by feature

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Andrew Dunstan wrote:
>
>
> Joshua D. Drake wrote:
>> I believe this patch is an update to the table_funcs contrib module.
>>
>>
>
> I spent 2 minutes looking. It has no Makefile and no comments. It
> doesn't use our code conventions either. At that stage I stopped looking.
>
> The author needs to spend some time looking at the developer
> documentation and perusing the lists to see what our requirements are if
> he wants to be taken seriously. Also, he needs to understand that the
> best way to go about such a project is to float ideas first, code later,
> not the other way around.

Woah there silver! Read the persons first post. He updated an existing
module to fit *his* needs. He then, instead of whining to the lists
about a missing feature, implemented what *he* needed.

It also appears that he fixed a few bugs *and* increased the usability
of the module. He then gave that code back. I say bravo and thanks for
the patch.

Yes the patch needs some work. I don't argue that but come on.

Example 1:

Son: Dad, I just took cleaned out the garage.
Dad: I see you used the wrong broom and didn't close the garage door

Example 2:

Son: Dad, I just cleaned out the garage.
Dad: Oh? Thanks, In the future use the shop broom and close the garage
door.

Sheesh, I know that everyone is trying to get the release out but let's
not remove our community building in the process.

Sincerely,

Joshua D. Drkae

>
> cheers
>
> andrew
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: Have you checked our extensive FAQ?
>
> http://www.postgresql.org/docs/faq
>

- --

=== The PostgreSQL Company: Command Prompt, Inc. ===
Sales/Support: +1.503.667.4564 24x7/Emergency: +1.800.492.2240
PostgreSQL solutions since 1997 http://www.commandprompt.com/
UNIQUE NOT NULL
Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
PostgreSQL Replication: http://www.commandprompt.com/products/

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGvclJATb/zqfZUUQRAuGfAJwMg3ZiZy8XBG8cIMea+X+JkXshAwCeKlmf
IYxJfh1FAOJ2k+bDt6QKOg4=
=o9VJ
-----END PGP SIGNATURE-----