public inbox for [email protected]  

help / color / mirror / Atom feed

From: Chapman Flack <[email protected]>
Subject: Re: [Pljava-dev] annotation keyword changes before 1.5.0 (?)
Date: Sun, 20 Dec 2015 21:49:50 -0500
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAKJ3sLy04vuLwnDNBrMnGAp=npj4R9x2cG10=Jc_6sLfBDFBnQ@mail.gmail.com>
References: <[email protected]>
	<CAKJ3sLy04vuLwnDNBrMnGAp=npj4R9x2cG10=Jc_6sLfBDFBnQ@mail.gmail.com>

On 12/20/15 20:11, Peter Henderson wrote:
> volatility=Volatile
> volatility=Stable
> volatility=Immutable
> 
> Make sense to me. Plus relate to the same terms used by a PL/pgSQL function.

Hi Peter,

I may have overlooked something; is there any context in PL/pgSQL where
you in fact say volatility=Volatile (or volatility=Stable, etc.)?

Or are you just referring to the standard CREATE FUNCTION syntax, where
you simply say VOLATILE, or STABLE, or IMMUTABLE?

I'm absolutely not proposing to change the names volatile, stable, and
immutable ... everybody knows those.

It just seems to me that the Java annotation syntax is adding a somewhat
unusual need to put somekeyword= in front of those three well-known names.
You don't ever have to do that in the SQL syntax, it's just a new slight
awkwardness imposed in the Java syntax, and my proposal is only about
what the somekeyword should be. It's effectively a non-optional noise
word, and to keep it unobtrusive I'd like it to be a short one. (I'm
sure that's the reason Thomas chose 'type' originally; it's nice and
short, and only unfortunate because it would be natural to use that
word to talk about the function's data type.)

Because nobody's changing the value names, even if the keyword were
Fahrvergnügen, you would still see Fahrvergnügen=IMMUTABLE or
Fahrvergnügen=VOLATILE and you'd immediately recognize what it was.

But 'effects' is even shorter than 'Fahrvergnügen', and almost as short
as 'type', without giving you the feeling of repeating yourself, as
volatility=VOLATILE would.

I flat-out refuse to make it Fahrvergnügen even if that gets the most votes.

-Chap
_______________________________________________
Pljava-dev mailing list
[email protected]
http://lists.pgfoundry.org/mailman/listinfo/pljava-dev

view thread (4+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected]
  Subject: Re: [Pljava-dev] annotation keyword changes before 1.5.0 (?)
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox