Forum OpenACS Improvement Proposals (TIPs): Re: TIP #37 (Proposed): Queries in XQL file conventions

Collapse
36: Re: TIP #37 (Proposed): Queries in XQL file conventions (response to 35)
Posted by Joel Aufrecht on 12/10/03 10:53 AM
1) If someone is on the OCT, their vote counts regardless of whether or not you agree with their reasons.  If you really think they're wrong, you need to get six OCT members to agree.

2) I think the original TIP was slightly unclear in that it did not describe the current situation fully.  The current situation is, I believe, that "best practice" is for all SQL to be in XQL files.  The veto means that this is still the case.

3) The status quo leaves two obvious questions which I don't believe were directly addressed:
  a) Will we reject packages that don't meet this best practice?  (No idea what the consensus would be - sounds like a good show-of-hands question in an OCT meeting)

  b) Will we change the kernel to enforce this best practice - ie, will we start ignoring any SQL in TCL files?  Argument for: enforces consistency.  Argument against: is a useful shortcut during development; requires lots of work with only a potential, future reward.  I think this one is a clear no, and status quo is "no," so unless someone starts a new TIP I don't think this issue is open.

Collapse
37: Re: TIP #37 (Proposed): Queries in XQL file conventions (response to 36)
Posted by Jeff Davis on 12/10/03 11:51 AM
For the record, the voting looks like: 
DaveB: no (?)
Don: no
Jeff: yes
Lars: yes
Roberto: ?
Peter: ?
Caroline: no
Tilmann: no
Malte: no
where ? means didn't vote and (?) means I think it was a no but I am not sure if it was no to the proposal or if it was just no to trying to fix any that currently exist (remember that the original proposal is simply one of coding standards i.e. explicitly stating that having generic queries in the .tcl file is acceptible).

Apologies if I got anyones vote wrong here.

In any case it seems like the required 2/3 is not there so this is rejected and the status quo coding standard: all queries in .xql files with the exception of things that cannot be multidb by design (schema browser for example).

It's also worth noting that the existing packages do not adhere to the existing standard very well in any case, and regardless of whether this is approved there is a lot of work to be done to bring things into line.