Re: Hi & some questions regarding Properties & contribution

From: Jürgen Rose <cptmauli(at)googlemail(dot)com>
To: Dave Cramer <pg(at)fastcrypt(dot)com>
Cc: List <pgsql-jdbc(at)postgresql(dot)org>
Subject: Re: Hi & some questions regarding Properties & contribution
Date: 2014-04-12 22:58:05
Message-ID: 5349C4FD.5000203@googlemail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

Hi Dave,

The code itself is finished, I still have to test it and I'm still
waiting on an answer from the OSGi mailinglist how properties should be
handled which are not there (ignore or exception).

About the description property, I was just curious. Is there actually an
up to date JDBC test suite somewhere? Do you test the driver for
conformity somehow?

The Eclipse contribution policy is really strict. You can find details
her: http://wiki.eclipse.org/Development_Resources/Contributing_via_Git
and here
http://www.eclipse.org/projects/dev_process/ip-process-in-cartoons.php

The Apache process is pretty similar:
http://www.apache.org/licenses/#clas thats why it is not a problem for
Eclipse to add dependencies based on apache projects. To provide a
different example - Chef:
http://docs.opscode.com/community_contributions.html and some more:
http://openmetaverse.org/about-open-metaverse-foundation
http://www.overte.org/contribution-agreement.html

It sounds like work, and it is work, and it makes it of course more
difficult to participate. But it definitely gives you also some security
that you don't run into legal problems.

Formatting:

I will not begin a formatting binge (-> ugly diffs). I just saw that in
some files the formatting was inconsistent in the file itself, hence my
question. Funnily enough, your example is exactly the style we use in
the Eclipse SCADA project. For the JDBC driver I try to create a
formatter which resembles closely the majority of the existing code,
I'll add the file as well.

I found this in the README.md under "Testing":
> You also need to test your changes with older JDKs. PgJDBC must
> support JDK5 ("Java 1.5") and newer, which means you can't use
> annotations, auto-boxing, for (:), and numerous other features added
> since JDK 5. Code that's JDBC4 specific may use JDK6 features, and
> code that's JDBC4.1 specific may use JDK7 features. Common code and
> JDBC3 code needs to stick to Java 1.5.
I would say, this paragraph is misleading. If you say PgJDBC must
support JDK5, then annotations, enhanced for loops, generics and so on
can be used! Only for JDBC2 this is relevant, but its not supported
anyway as far as I understand the build. The only thing which one has to
be really careful are things which were added after Java 1.5 (e.g.
override annotations on interfaces, switch with strings etc).

best regards
Jürgen

Am 12.04.2014 01:16, schrieb Dave Cramer:
> Jurgen,
>
> Implement properties as you see fit. With docs for them as well
>
> There is no specific reason for not having the description property.
>
> What is their contribution policy? Where do I find it?
>
> Formatting the code en-masse will be a problem. If you want to format
> your new files as per eclipse be my guest.
>
> My basic rule is keep the formatting the same as whatever file you are
> editing is. Formatting is a religious argument.
>
> If I had my druthers formatting would be have lots of whitespace, and
> others don't usually agree
>
> ie function( int arg )
> {
> if ( something == something )
> {
> do something
> }
> }
>
>
> as for header information it should have
>
> Copyright (c) 2004-2014, PostgreSQL Global Development Group */
>
> Feel free to fix headers in any other files you touch.
>
>
> Thanks!!!
>
> Dave Cramer
>
> dave.cramer(at)credativ(dot)ca
> http://www.credativ.ca
>
>
> On 11 April 2014 18:00, Jürgen Rose <cptmauli(at)googlemail(dot)com
> <mailto:cptmauli(at)googlemail(dot)com>> wrote:
>
> Hi,
>
> I'm Jürgen, new on this list and I'm currently osgifying the
> postgresql
> jdbc driver (https://github.com/pgjdbc/pgjdbc/issues/71)
>
> I have some questions regarding the conformity to the JDBC spec.
>
> In section 9.4.1 "DataSource Properties" it reads: The only property
> required for all DataSource implementations is description
>
> When I look at the implementation, there is no description
> property. Is
> there any specific reason for this?
>
> The spec also defines a number of other (optional) properties, most of
> them are not implemented. Wouldn't it be possible to implement at
> least
> some of them (dataSourceName, ...), just to be nice?
>
> I'm also talking with the IP people from the Eclipse foundation,
> so that
> the JDBC driver can be used as a dependency in some of the projects.
> This is not possible at the moment since it failed the validation (the
> Eclipse foundation is pretty strict). I'm still trying to find out
> what
> the specific problems are, but as far as I understand a contribution
> policy is missing (and maybe other things). And actually I didn't find
> one when I looked.
>
> Two things more, how to format the source code? It seems to me
> that the
> code is formatted relatively inconsistent. I'd love to have a
> formatter
> for Eclipse.
>
> What about the header information in the files? Some files have none,
> some have /* Copyright (c) 2004-2011, PostgreSQL Global Development
> Group */. What should I put into my new file?
>
>
> Jürgen
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org
> <mailto:pgsql-jdbc(at)postgresql(dot)org>)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>
>

In response to

Responses

Browse pgsql-jdbc by date

  From Date Subject
Next Message Dave Cramer 2014-04-12 23:32:59 Re: Hi & some questions regarding Properties & contribution
Previous Message Dave Cramer 2014-04-11 23:16:54 Re: Hi & some questions regarding Properties & contribution