Re: Transaction Status Python Tests

From: Yosry Muhammad <yosrym93(at)gmail(dot)com>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Dave Page <dpage(at)pgadmin(dot)org>
Subject: Re: Transaction Status Python Tests
Date: 2019-08-19 12:20:08
Message-ID: CAFSMqn8tk=V_tFWifWfo0Xfhcqp3x7eFWbWRdtjOooF6sKOLVw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Can you send me the error message? Unfortunately there is a problem with my
machine right now I am unable to investigate it myself, I am very sorry for
the inconvenience.

On Mon, Aug 19, 2019, 2:08 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi Yosry
>
> We have tried a lot to figure out the issue but didn't find the actual
> cause. Can you please figure out and send the patch asap, I'll have to
> create candidate build tomorrow.
>
> On Mon, Aug 19, 2019 at 1:22 PM Yosry Muhammad <yosrym93(at)gmail(dot)com> wrote:
>
>> Hi Akshay,
>>
>> I am sorry I do not have access to my machine right now. I will try to
>> work on it (and the JS patch) once I get the chance to.
>>
>> On Mon, Aug 19, 2019, 9:49 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
>> wrote:
>>
>>> Hi Yosry
>>>
>>> Test cases are failing on our build machines. When I reviewed your patch
>>> I have run test cases with paramaster "*--pkg tools.sqleditor.tests*"
>>> on multiple database servers and every time test case has been passed.
>>>
>>> But on build machines we run test cases with parameter "*--pkg all* *--exclude
>>> feature_tests" *on multiple database server and it fails for second
>>> server onwards. I have also tried the same and it is failing on my machine
>>> too.
>>>
>>> I am debugging the code why it is getting failed, but if you found any
>>> solution please send the patch asap.
>>>
>>> On Mon, Aug 19, 2019 at 11:48 AM Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Thanks patch applied.
>>>>
>>>> On Sat, Aug 17, 2019 at 6:22 PM Yosry Muhammad <yosrym93(at)gmail(dot)com>
>>>> wrote:
>>>>
>>>>> Hi Hackers,
>>>>>
>>>>> Please find attached a patch including new python test that checks the
>>>>> transaction status after executing queries and saving data when auto-commit
>>>>> is on/off. It also includes some refactoring of previously written python
>>>>> tests.
>>>>>
>>>>> Please review !
>>>>> Thanks. 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/
>>>>>
>>>>
>>>>
>>>> --
>>>> *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*
>>>
>>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2019-08-19 12:29:59 Re: Transaction Status Python Tests
Previous Message Akshay Joshi 2019-08-19 12:08:06 Re: Transaction Status Python Tests