Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:98360 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 71402 invoked from network); 28 Feb 2017 04:14:57 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 28 Feb 2017 04:14:57 -0000 Authentication-Results: pb1.pair.com header.from=christopher.jones@oracle.com; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=christopher.jones@oracle.com; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain oracle.com from 141.146.126.69 cause and error) X-PHP-List-Original-Sender: christopher.jones@oracle.com X-Host-Fingerprint: 141.146.126.69 aserp1040.oracle.com Received: from [141.146.126.69] ([141.146.126.69:31118] helo=aserp1040.oracle.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id E4/65-62026-A39F4B85 for ; Mon, 27 Feb 2017 23:14:55 -0500 Received: from aserv0021.oracle.com (aserv0021.oracle.com [141.146.126.233]) by aserp1040.oracle.com (Sentrion-MTA-4.3.2/Sentrion-MTA-4.3.2) with ESMTP id v1S4Elnq012831 (version=TLSv1 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 28 Feb 2017 04:14:47 GMT Received: from aserv0122.oracle.com (aserv0122.oracle.com [141.146.126.236]) by aserv0021.oracle.com (8.13.8/8.14.4) with ESMTP id v1S4Ek8h007892 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK); Tue, 28 Feb 2017 04:14:47 GMT Received: from abhmp0007.oracle.com (abhmp0007.oracle.com [141.146.116.13]) by aserv0122.oracle.com (8.14.4/8.14.4) with ESMTP id v1S4EkaI032054; Tue, 28 Feb 2017 04:14:46 GMT Received: from [10.191.131.110] (/10.191.131.110) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Mon, 27 Feb 2017 20:14:46 -0800 To: Adam Baratz References: Cc: "internals@lists.php.net" Message-ID: <98c2a0bb-c03a-a1ba-75f2-b6210709aa81@oracle.com> Date: Tue, 28 Feb 2017 15:14:43 +1100 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Source-IP: aserv0021.oracle.com [141.146.126.233] Subject: Re: [PHP-DEV] Re: [RFC] Extended String Types For PDO From: christopher.jones@oracle.com (Christopher Jones) On 26/2/17 9:07 am, Marco Pivetta wrote: > Hi Adam > > On Fri, Feb 24, 2017 at 2:28 PM, Adam Baratz wrote: > >>> Based on some pain points with my team and things I've heard from others, >>> I created an RFC to handle "national" character sets for emulated >> prepared >>> statements: >>> https://wiki.php.net/rfc/extended-string-types-for-pdo >>> >>> I had previously suggested this as a driver-specific change, but believe >>> it's worthwhile to make it generic since it affects multiple drivers: >>> https://externals.io/thread/400#email-12542 >>> >>> Please let me know what you think. >>> >>> Thanks, >>> Adam >>> >> Any thoughts on this one? >> > DBAL maintainer here: before we introduce even more complexity into the PDO > stuff (which is already a maze), could a set of test cases be written, so > that stuff that is currently impossible to do without this RFC is > clearer/demonstrated? > > I'm asking because we didn't get any bug reports about extended string > types for Doctrine DBAL, and adding new types just to workaround the > limitations of the usual suspects (remember that PDO for SQLServer is > experimental, if not totally unusable) is shotgun surgery, and just more > complexity to handle. > > Greets, > > Marco Pivetta > > http://twitter.com/Ocramius > > http://ocramius.github.com/ > I agree the spec could definitely do with some samples and detail. At a start, what about defining 'N-prefix' so the reader doesn't have to trawl through references?! And clarify whether the PR is for the generic solution or the earlier problem. More importantly, what about allowing arbitrary quote delimiters? https://livesql.oracle.com/apex/livesql/file/content_CIREYU9EA54EOKQ7LAMZKRF6P.html Chris -- http://twitter.com/ghrd