[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

casing of SQL statements? (was: svn commit: r1000370...)

From: Greg Stein <gstein_at_gmail.com>
Date: Thu, 23 Sep 2010 13:44:22 -0400

On Thu, Sep 23, 2010 at 13:41, Philip Martin <philip.martin_at_wandisco.com> wrote:
> Greg Stein <gstein_at_gmail.com> writes:
>
>> On Thu, Sep 23, 2010 at 05:16,  <philip_at_apache.org> wrote:
>>> Author: philip
>>> Date: Thu Sep 23 09:16:28 2010
>>> New Revision: 1000370
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1000370&view=rev
>>> Log:
>>> Use lower case in SQL queries instead of mixing upper and lower
>>> cases inconsistently.  No functional change.
>>
>> I preferred the form with uppercase keywords and table names. It makes
>> it much easier to see which columns are selected, and to read the
>> expressions in WHERE clauses.
>>
>> IOW, I disagree with this change and would actually prefer fixing the
>> all-lowercase queries that were scattered around.
>
> I don't mind whether we use upper case or lower case, I just want all
> keywords in the same case, all table names in the same case and all
> column names in the same case (and those 3 categories don't have to be
> the same).  I chose lower case simply because there appeared to be
> more lower case than uppercase.  I'm prepared to make the change to
> upper case if that is what people want.  Or you can do it :)

I'm fine with doing the work. Let's gather a few preferences, and if
they support a change from all-lower-case, then I'll make it happen.

People?

Cheers,
-g
Received on 2010-09-23 19:45:15 CEST

This is an archived mail posted to the Subversion Dev mailing list.