Hello Hackers,
It's related to applying data sorting on table data by primary key.
With the existing implementation, we can view the table's data using 4
options with the different orders by default
1 - All Rows (No order)
2 - First 100 rows (ASC order)
3 - Last 100 rows (DESC order)
4 - Filtered rows (No order)
In the https://redmine.postgresql.org/issues/5157 it's not clearly
mentioned on which option to apply sorting by PK? I'm assuming that should
be on ALL Rows option.
Please suggest.
Thanks!
On Tue, Apr 21, 2020 at 10:12 AM navnath gadakh <
navnath.gad...@enterprisedb.com> wrote:
> Hi Khushboo,
> Please hold this patch for review I'm still optimizing the code in the
> patch.
>
>
> On Mon, Apr 20, 2020 at 9:16 PM navnath gadakh <
> navnath.gad...@enterprisedb.com> wrote:
>
>> Hi Khushboo,
>> I have modified the code as per review comments. Please review the
>> attached patch file.
>>
>> Thanks!
>>
>> On Mon, Apr 20, 2020 at 10:56 AM Khushboo Vashi <
>> khushboo.va...@enterprisedb.com> wrote:
>>
>>> Hi Navnath,
>>>
>>> Review comments:
>>>
>>> 1. If we have multiple Primary keys, then we should include all the keys
>>> into the Order by clause.
>>> 2. In the Preferences dialog, please put this option in the Query Tool
>>> > Options instead of Result Grid and also change the Label.
>>> 3. Please optimize the code, as I can see objectname.sql file is being
>>> used in else condition also, which is not required. Based on the parameter
>>> setting, Just one call of that sql is enough.
>>>
>>> Thanks,
>>> Khushboo
>>>
>>>
>>> On Fri, Apr 17, 2020 at 6:43 PM navnath gadakh <
>>> navnath.gad...@enterprisedb.com> wrote:
>>>
>>>> Hello Hackers,
>>>>
>>>> Please find the modified patch with an option in Preferences for data
>>>> sorting by the primary key. Also, the previous patch was not working with
>>>> table has no primary key.
>>>>
>>>> Thanks!
>>>>
>>>>
>>>>
>>>>
>>>> On Thu, Apr 16, 2020 at 5:01 PM Dave Page <dave.p...@enterprisedb.com>
>>>> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Thu, Apr 16, 2020 at 12:08 PM navnath gadakh <
>>>>> navnath.gad...@enterprisedb.com> wrote:
>>>>>
>>>>>> Hi Dave/Team,
>>>>>> This patch is related to the default sort order for the
>>>>>> view table data. In pgAdminIII default ordering is by primary key and
>>>>>> this
>>>>>> is not working in pgAdminIV.
>>>>>> I have attached the patch with the back end code.
>>>>>>
>>>>>> Please review it.
>>>>>>
>>>>>> *Question*: There is one suggestion on
>>>>>> https://redmine.postgresql.org/issues/5157 about to put a checkbox
>>>>>> in the configuration for this behavior.
>>>>>> Do I need to implement that really? I yes, Is
>>>>>> preferences a good place for that? / Suggestions?
>>>>>>
>>>>>
>>>>> I think we should make this optional, and yes, Preferences is a good
>>>>> place. The reason is that sorting data is not without cost - at the very
>>>>> least it will require use of an index to access what may be the whole
>>>>> table.
>>>>>
>>>>> --
>>>>> Dave Page
>>>>> VP & Chief Architect, Database Infrastructure
>>>>> EnterpriseDB: http://www.enterprisedb.com
>>>>> The Enterprise PostgreSQL Company
>>>>>
>>>>> Blog: http://pgsnake.blogspot.com
>>>>> Twitter: @pgsnake
>>>>>
>>>>
>>>>
>>>> --
>>>> Regards,
>>>> Navnath Gadakh
>>>>
>>>
>>
>> --
>> Regards,
>> Navnath Gadakh
>>
>
>
> --
> Regards,
> Navnath Gadakh
>
--
Regards,
Navnath Gadakh
