[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Call for testing experimental patch



On second thought, there is no consistant naming in the current db so
it doesn;t matter.  I will go ahead and rename it.

On 9/20/06, Chris Travers <..hidden..> wrote:
I suppose if people would rather have it called "transactions" I can
rename it transactions.

Best Wishes,
Chris Travers

On 9/20/06, Joshua D. Drake <..hidden..> wrote:
> Chris Travers wrote:
> > On 9/19/06, David Tangye <..hidden..> wrote:
> >> On Tue, 2006-09-19 at 13:27 -0700, Chris Travers wrote:
> >>> Hi Tony,
> >>>
> >>> I think it is too late to change the name of the table to
> >>> transactions, but I will consider it.
> >> Its never too late to correct something, *if* its not right, surely?
> >
> > You have a point.  But I don't think this is at this point an issue of
> > one being "right" and the other being "not right" at the moment.  In
> > this case, I think it is a matter of stylistics.  Both names are
> > adequate and adequately descriptive, and both names have some slight
> > shortcomings.
>
> Well this really is semantics. We are essentially dealing with a ledger
> there for having ledger in the transactions name is redundant. It
> reminds me of people that do this:
>
> tbl_foo
> view_bar
>
> Which is silly.
>
> Joshua D. Drake
>
>
>
>
> >> Hmm, put another way: Ouch, isn't this sounding a tad like the sort of
> >> reaction we did not like when it came out of the SL camp?
> >
> > If you make a case that it is broken, we will fix it :-)  Right now,
> > this is a matter of whether we have consistant style and type a few
> > extra characters or whether we shorten the name a bit and change the
> > style of the rest of the db as we rewrite it.  I think it is too late
> > to revisit this sort of decision unless a case can be made that it is
> > materially wrong.  Otherwise, we can stop coding while we debate in
> > minute detail the arbitrary decisions made in the coding standards.
> > :-)
> >
> > Best Wishes,
> > Chris Travers
> >
> > -------------------------------------------------------------------------
> > Take Surveys. Earn Cash. Influence the Future of IT
> > Join SourceForge.net's Techsay panel and you'll get the chance to share your
> > opinions on IT & business topics through brief surveys -- and earn cash
> > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> > _______________________________________________
> > Ledger-smb-devel mailing list
> > ..hidden..
> > https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel
> >
>
>
> --
>
>     === The PostgreSQL Company: Command Prompt, Inc. ===
> Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240
>     Providing the most comprehensive  PostgreSQL solutions since 1997
>               http://www.commandprompt.com/
>
>
>
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys -- and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Ledger-smb-devel mailing list
> ..hidden..
> https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel
>