public inbox for [email protected]
help / color / mirror / Atom feedFrom: Akshay Joshi <[email protected]>
To: Dave Page <[email protected]>
Cc: Yosry Muhammad <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [GSoC] Support editing resultsets from tables with OIDs
Date: Mon, 29 Jul 2019 12:37:47 +0530
Message-ID: <CANxoLDdmjbmYMEKQT3qAKx+LxqSy8zLum+WAzda-vz_P9j2PwQ@mail.gmail.com> (raw)
In-Reply-To: <CANxoLDfpbRJxWrB8BFY=pQ=8FMDW8hL5TSEDaE1q_XvB2N_kcw@mail.gmail.com>
References: <CAFSMqn-YcFFUdSApFn6-=CeGFtARRJNkZv+tGJ7ORMe7ahftEQ@mail.gmail.com>
<CA+OCxowy7yP8SY=tLMkJ7yLwPt+n=Etww9qhzOtjLqW3EakHDA@mail.gmail.com>
<CANxoLDfpbRJxWrB8BFY=pQ=8FMDW8hL5TSEDaE1q_XvB2N_kcw@mail.gmail.com>
Thanks patch applied.
On Mon, Jul 29, 2019 at 11:12 AM Akshay Joshi <[email protected]>
wrote:
>
> On Fri, Jul 26, 2019 at 5:33 PM Dave Page <[email protected]> wrote:
>
>> Akshay; can you take a look at this and commit if appropriate please? I'm
>> somewhat busy at the moment :-(
>>
>
> Sure, I am looking it.
>
>>
>> On the subject of the regression test - I assume at one point the
>> messages window was including the command tags in it's output. This failure
>> would indicate we broken that. Anyone got any ideas how that happened?
>>
>> On Thu, Jul 25, 2019 at 1:01 PM Yosry Muhammad <[email protected]>
>> wrote:
>>
>>> Hi,
>>> Please find a patch attached to add supporting of resultsets from tables
>>> with OIDs as updatable resultsets (with updated test cases and docs) - this
>>> is a part of a GSoC project. Please review !
>>> This patch also includes a small refactor of the previous patch and a
>>> small bug fix that occurred when the query resultset had no rows but was
>>> updatable (row additions were not permitted, although they should be).
>>>
>>>
>>> On a different note, one of the feature tests fails (on master),
>>> probably due to a small typo. In query_tool_tests.py this function call:
>>>
>>> self.page.find_by_xpath(
>>> '//div[contains(@class, "sql-editor-message") and '
>>> 'contains(string(), "CREATE TABLE")]'
>>> )
>>>
>>> on lines 371-374 produces an exception that the element cannot be found,
>>> this is probably due to the fact that the message returned by the query
>>> executed before this call does not contain the string "CREATE TABLE". Just
>>> letting you know about this.
>>>
>>> Thanks and regards!
>>> --
>>>
>>> *Yosry Muhammad Yosry*
>>>
>>> Computer Engineering student,
>>> The Faculty of Engineering,
>>> Cairo University (2021).
>>> Class representative of CMP 2021.
>>> https://www.linkedin.com/in/yosrym93/
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
view thread (6+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: [GSoC] Support editing resultsets from tables with OIDs
In-Reply-To: <CANxoLDdmjbmYMEKQT3qAKx+LxqSy8zLum+WAzda-vz_P9j2PwQ@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox