[mapguide-internals] RE: MapGuide RFC 82 is posted for review

Klain Qin klain.qin at autodesk.com
Wed Aug 26 02:12:14 EDT 2009


Thanks Trevor. The RFC has been updated based on your comments. 

http://trac.osgeo.org/mapguide/wiki/MapGuideRfc82

And are there any other comments on this RFC? If not, I'd like to ask for a vote on it?

Thanks,
Klain

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Trevor Wekel
Sent: Wednesday, August 26, 2009 2:45 AM
To: MapGuide Internals Mail List
Subject: [mapguide-internals] RE: MapGuide RFC 82 is posted for review

Hi Klain,

The RFC looks ok but the Doxygen syntax contained in the comments needs to be updated to reflect the correct function signatures.  You may also want to specifically mention all of the derived classes which will implement the new signatures (MgProxyFeatureReader, MgProxyDataReader, MgServerDataReader, MgServerFeatureReader, etc).

Thanks,
Trevor

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Klain Qin
Sent: August 13, 2009 10:28 PM
To: MapGuide Internals Mail List
Subject: [mapguide-internals] MapGuide RFC 82 is posted for review

Hi All,

Please review MapGuide RFC 82 - Extend Reader Access By Index. Any comments are much appreciated. 

http://trac.osgeo.org/mapguide/wiki/MapGuideRfc82

Thanks,
Klain

-----Original Message-----
From: mapguide-internals-bounces at lists.osgeo.org [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Leaf Li
Sent: Friday, August 14, 2009 9:41 AM
To: mapguide-internals at lists.osgeo.org
Subject: [mapguide-internals] MapGuide RFC 80 is updated.

All,

I update MapGuide RFC 80. Can you review it?

The original solution is to add two new overloaded method ExecuteSqlQuery and ExecuteNoSqlQuery to support parameter binding. The current plan is to extend method ExecuteSqlQuery and ExecuteNoSqlQuery added in MapGuide RFC 78 because those two methods isn't released yet.

Thanks,
Leaf Li
_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals


_______________________________________________
mapguide-internals mailing list
mapguide-internals at lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/mapguide-internals


More information about the mapguide-internals mailing list