RE: Code Review Process

From: Herring Dave - dherri <Dave.Herring_at_acxiom.com>
Date: Fri, 8 Feb 2013 22:10:10 +0000
Message-ID: <BD475CE0B3EE894DA0CAB36CE2F7DEB401045F4BC8_at_LITIGMBCRP02.Corp.Acxiom.net>

Jeff,

For the most part we're getting 1 to many scripts to run as a change and to blindly run them, sending the developers back all generated logs (we use a wrapper for every script so capture is the same every time) and updating the change ticket with that info. I try to spot check as sometimes they try creating tablespaces and make rather poor choices or try doing a "GRANT ALL" instead of listing the privs they really need. But for the most part there's no way we'd have time to check all code when we're on "change request" duty.

DAVID HERRING
DBA
Acxiom Corporation

EML   dave.herring_at_acxiom.com
TEL    630.944.4762
MBL   630.430.5988 

1501 Opus Pl, Downers Grove, IL 60515, USA WWW.ACXIOM.COM -----Original Message-----
From: oracle-l-bounce_at_freelists.org [mailto:oracle-l-bounce_at_freelists.org] On Behalf Of Jeff Chirco Sent: Wednesday, February 06, 2013 3:39 PM To: oracle-l_at_freelists.org
Subject: Code Review Process

What is everyone's code review process?
* Do you review code during development or right before going to QA?

• Or do developers come to you and say we need this to go to production right now can you check it out really quick?
• Are you proactive in the code review process or is there a formal submittal?

And what is your code review process like?

• Do you just check for naming standards? Use a special toll or application?
• Do you do some kind of performance testing on the code? Do you do it on all code or just some

Side question, which team writes your pl/sql code? Do have you dedicated pl/sql developers or do your Windows/Java/Cobol developers do the pl/sql as well? Our DBA team is very small so our .Net and Cobol programmers write most of their own pl/sql and since it is not their primary language it is not always the most efficient. So I am often reviewing it at the last minute with occasionally not much time to really make changes. I try to teach them as much as I can.

Some background, we write all our own applications and it is all for internal use .

Thanks in advance for any input.

--
http://www.freelists.org/webpage/oracle-l


***************************************************************************
The information contained in this communication is confidential, is
intended only for the use of the recipient named above, and may be legally
privileged.

If the reader of this message is not the intended recipient, you are
hereby notified that any dissemination, distribution or copying of this
communication is strictly prohibited.

If you have received this communication in error, please resend this
communication to the sender and delete the original message or any copy
of it from your computer system.

Thank You.
****************************************************************************

--
http://www.freelists.org/webpage/oracle-l

Received on Fri Feb 08 2013 - 23:10:10 CET