public inbox for [email protected]  

help / color / mirror / Atom feed

From: Aditya Toshniwal <[email protected]>
To: Victoria Henry <[email protected]>
Cc: Dave Page <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.
Date: Wed, 23 May 2018 10:13:27 +0530
Message-ID: <CAM9w-_mTe0gW+2cdhMutMWyBsRt7Om3G_Em126Ryntf0nOiHbA@mail.gmail.com> (raw)
In-Reply-To: <CANxYE3+qrGeZYzdYEtzxS8i2P+325V5E+n1xVpyzTwn7vRqRDg@mail.gmail.com>
References: <CAM9w-_ntCUguZp2rVJXPsDvWJbWneAz4-AHJoYB+nUTSYMXc9g@mail.gmail.com>
	<CA+OCxozRjJY6RWrXpMLD6-dWnm2LdPs6bS+VmJa403vxVoLwhg@mail.gmail.com>
	<CAM9w-_m-cJax16zqBonx-d0-HSy5dNoOzB13HWAad5Xv0Wu=CA@mail.gmail.com>
	<CANxYE3+qrGeZYzdYEtzxS8i2P+325V5E+n1xVpyzTwn7vRqRDg@mail.gmail.com>

Thank you Victoria, Anthony.

Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

On Tue, May 22, 2018 at 7:15 PM, Victoria Henry <[email protected]> wrote:

> Hi Aditya,
>
> We made a minor change to make the patch so the python linter can pass.
> Attached is the change we made.
> Everything else looks good.
>
> Sincerely,
>
> Victoria & Anthony
>
> On Tue, May 22, 2018 at 4:46 AM Aditya Toshniwal <aditya.toshniwal@
> enterprisedb.com> wrote:
>
>> Hi,
>>
>> PFA updated patch. Linter issues are fixed ( we dont have any linter
>> setup for python :-( )
>> Regarding test cases, they run successfully on my system and the reason
>> it failed for pivotal is timeout exception. I am sorry I can't help with
>> that.
>>
>> Traceback (most recent call last):
>>   File "/tmp/build/a453582b/pgadmin-repo/web/pgadmin/feature_
>> tests/keyboard_shortcut_test.py", line 52, in runTest
>>     self._check_shortcuts()
>>   File "/tmp/build/a453582b/pgadmin-repo/web/pgadmin/feature_
>> tests/keyboard_shortcut_test.py", line 77, in _check_shortcuts
>>     ") and contains(@class, 'open')]")
>>   File "/root/.pyenv/versions/pgadmin36/lib/python3.6/site-
>> packages/selenium/webdriver/support/wait.py", line 80, in until
>>     raise TimeoutException(message, screen, stacktrace)
>> selenium.common.exceptions.TimeoutException: Message:
>>
>> Thanks and Regards,
>> Aditya Toshniwal
>> Software Engineer | EnterpriseDB Software Solutions | Pune
>> "Don't Complain about Heat, Plant a tree"
>>
>> On Tue, May 22, 2018 at 1:37 PM, Dave Page <[email protected]> wrote:
>>
>>> Hi
>>>
>>> Pivotal's buildbot is showing problems with this patch:
>>>
>>> https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/
>>> pipelines/pgadmin-patch/jobs/run-linter/builds/66 (linter failed)
>>> https://gpdb-dev.bosh.pivotalci.info/teams/pgadmin/
>>> pipelines/pgadmin-patch/jobs/run-tests/builds/84 (tests failed)
>>>
>>>
>>> On Tue, May 22, 2018 at 7:05 AM, Aditya Toshniwal <aditya.toshniwal@
>>> enterprisedb.com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> PFA patch for RM#3289 where decode error was thrown on querying a
>>>> SQL_ASCII database table. Please note, this problem occurs only on windows.
>>>> Sample insert - insert into test_tab values ('é');
>>>>
>>>> psycopg2 has a encodings dictionary where Postgres Database Encodings
>>>> are mapped to python equivalent. It uses 'ascii' decoder of python to
>>>> decode for SQL_ASCII encoding. If data has characters beyond the limit of
>>>> ascii then it failed. The solution would be to use utf_8 decoder instead of
>>>> ascii. I tried setting the client_encoding using
>>>> set_client_encoding('UTF8') method of a psycopg2 connection but no luck
>>>> (also its not allowed for async connection). I also tried executing "SET
>>>> CLIENT_ENCODING='UTF8'" but it didn't work too.
>>>> So, as in the patch, I had to set encodings dict value directly to
>>>> 'utf_8' and it seems to be working. Please note, the same is added to
>>>> psycopg3 milestones
>>>> https://github.com/psycopg/psycopg2/milestone/4
>>>>
>>>> Also fixed a small glitch for sql editor connection status check.
>>>>
>>>> Kindly review.
>>>>
>>>> Thanks and Regards,
>>>> Aditya Toshniwal
>>>> Software Engineer | EnterpriseDB Software Solutions | Pune
>>>> "Don't Complain about Heat, Plant a tree"
>>>>
>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>

view thread (38+ 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: [pgAdmin4][RM#3289] Can't query SQL_ASCII database.
  In-Reply-To: <CAM9w-_mTe0gW+2cdhMutMWyBsRt7Om3G_Em126Ryntf0nOiHbA@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