MapGuide RFC 80 is updated.

[Leaf Li]

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
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals

[Klain Qin]

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: [hidden email] [mailto:[hidden email]] On Behalf Of Leaf Li
Sent: Friday, August 14, 2009 9:41 AM
To: [hidden email]
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
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals

[Leaf Li]

All,

Currently, MgParameter is similar to MgNullableProperty. The only difference between them is that MgParameter has one additional property "parameter direction". So according to Bruce's suggestion, I update MapGuide RFC 80 to construct an MgParameter by an MgNullableProperty instance and an MgParameterDirection value.

Can you review it?
http://trac.osgeo.org/mapguide/wiki/MapGuideRfc80

Thanks,
Leaf Li
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals

[Trevor Wekel]

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: [hidden email] [mailto:[hidden email]] 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: [hidden email] [mailto:[hidden email]] On Behalf Of Leaf Li
Sent: Friday, August 14, 2009 9:41 AM
To: [hidden email]
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
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals


_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals

[Klain Qin]

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: [hidden email] [mailto:[hidden email]] 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: [hidden email] [mailto:[hidden email]] 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: [hidden email] [mailto:[hidden email]] On Behalf Of Leaf Li
Sent: Friday, August 14, 2009 9:41 AM
To: [hidden email]
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
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals


_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals
_______________________________________________
mapguide-internals mailing list
[hidden email]
http://lists.osgeo.org/mailman/listinfo/mapguide-internals