Opened 14 years ago

Closed 14 years ago

#397 closed enhancement (fixed)

Replace GetSiteInformation with GetSiteVersion

Reported by: trevorwekel Owned by: madair
Priority: P2 Milestone: 2.2
Component: MapGuide Version: 2.0.0-RC1
Severity: Major Keywords:
Cc: brucedechant Browser: All
External ID: Operating System: All
state: New

Description

The call to GetSiteInformation for MapGuide layers is expensive and performs poorly in multiuser environments with the recent mutex addition to the MapGuide Server. Replacing the call with GetSiteVersion is much faster and more scalable.

Attachments (1)

GetSiteVersion.patch (552 bytes ) - added by trevorwekel 14 years ago.

Download all attachments as: .zip

Change History (9)

by trevorwekel, 14 years ago

Attachment: GetSiteVersion.patch added

comment:1 by brucedechant, 14 years ago

Cc: brucedechant added

comment:2 by madair, 14 years ago

Status: newassigned

it looks like the PHP GetSiteVersion method is only available in a recent version MGOS? (I'm testing with 2.1.0) If so, we can't put this into the 2.0 branch because of backwards compatibility issues.

comment:3 by brucedechant, 14 years ago

The GetSiteVersion was added to MGOS 2.2.

Is it possible to put this patch in the fusion trunk so that future releases of MGOS can benefit from this fix?

comment:4 by trevorwekel, 14 years ago

Ok. Can we put the change into trunk and then create a branch or sandbox for MapGuide 2.2 from trunk? The MapGuide 2.2 Beta was released from trunk revision 2155. The Fusion 2.0 branch will not work with MapGuide 2.2 due to API changes.

comment:5 by madair, 14 years ago

yes we can put it in trunk, but I'll also keep a version of Utilities.php that will work with older installations. It also means the 2.0 branch won't be suitable for MGOS2.2

comment:6 by trevorwekel, 14 years ago

Ok. That's good. Due to http://trac.osgeo.org/mapguide/wiki/MapGuideRfc89, MGOS2.2 is not API compatible with MGOS2.1. This gets reflected in the various Fusion PHP scripts so we need a new branch anyway.

comment:7 by madair, 14 years ago

Milestone: 2.02.2

changing milestone to 2.2 since this breaks backward compatibility

comment:8 by madair, 14 years ago

Resolution: fixed
Status: assignedclosed

fixed at rev 2187 - trunk is not longer compatible with MGOS versions prior to 2.2

Note: See TracTickets for help on using tickets.