CDR Tickets

Issue Number 3183
Summary [Internal] Modify Publishing.py Module
Created 2010-06-24 15:48:40
Issue Type Improvement
Submitted By Englisch, Volker (NIH/NCI) [C]
Assigned To Kline, Bob (NIH/NCI) [C]
Status Closed
Resolved 2010-06-25 15:36:27
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.107511
Description

BZISSUE::4870
BZDATETIME::2010-06-24 15:48:40
BZCREATOR::Volker Englisch
BZASSIGNEE::Bob Kline
BZQACONTACT::Volker Englisch

We want to modify the publishing.py module in order to use the cdrdb.py module for connecting to the database instead of using the ADODB connection.
This change will likely speed up the time it takes to access the publishing UI.

Comment entered 2010-06-24 16:55:07 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2010-06-24 16:55:07
BZCOMMENTOR::Volker Englisch
BZCOMMENT::1

I've made the change to the program without a significant speed increase.
publishing.py - R9721

Bob would also like to try replacing the xml.dom.minidom module with lxml.etree, so I'm reassigning the task to him.

Comment entered 2010-06-25 10:43:41 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2010-06-25 10:43:41
BZCOMMENTOR::Bob Kline
BZCOMMENT::2

I rewrote the code which constructs the main menu of publishing systems to eliminate the delay, which sometimes took so long that it timed out on the production server. The new code is installed on Mahler. Give it a try, Volker.

Comment entered 2010-06-25 11:11:38 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2010-06-25 11:11:38
BZCOMMENTOR::Volker Englisch
BZCOMMENT::3

You basically removed the function __getPubSys() with its inner select SQL statement which was probably causing the delay, right?

It certainly speeds things up dramatically!!! I like it.

Little drawback though: The link to PubStats.py (Manage Publishing Job Status) isn't working.

Comment entered 2010-06-25 11:58:30 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2010-06-25 11:58:30
BZCOMMENTOR::Bob Kline
BZCOMMENT::4

(In reply to comment #3)

> The link to PubStats.py (Manage Publishing Job Status) isn't working.

Oops! Typo fixed. Please give it another shot.

Comment entered 2010-06-25 14:56:27 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2010-06-25 14:56:27
BZCOMMENTOR::Volker Englisch
BZCOMMENT::5

The link is working again.

I ran a test Hotfix-Export and everything appears to be working properly (and fast).

Comment entered 2010-06-25 15:23:04 by Kline, Bob (NIH/NCI) [C]

BZDATETIME::2010-06-25 15:23:04
BZCOMMENTOR::Bob Kline
BZCOMMENT::6

Promoted to Bach and Franck.

Comment entered 2010-06-25 15:36:27 by Englisch, Volker (NIH/NCI) [C]

BZDATETIME::2010-06-25 15:36:27
BZCOMMENTOR::Volker Englisch
BZCOMMENT::7

Sweet! Closing issue.

Elapsed: 0:00:00.001660