Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree

От

Ashesh Vashi

Тема

Дата

11 мая 2018 г. в 09:31:53

Msg-id

CAG7mmowkNtkoLGQFeWAH-TSOi+QWbHT2-RbMmjRDAMAxMufmeg@mail.gmail.com

обсуждение

Ответ на

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree (Anthony Emengo)

Список

pgadmin-hackers

Дерево обсуждения

[pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 2 апреля 2018 г. в 21:45:32

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 4 апреля 2018 г. в 19:17:01

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Khushboo Vashi <khushboo.vashi@enterprisedb.com> 5 апреля 2018 г. в 06:35:40

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Khushboo Vashi <khushboo.vashi@enterprisedb.com> 5 апреля 2018 г. в 13:31:48

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 5 апреля 2018 г. в 16:38:31

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> 6 апреля 2018 г. в 11:48:21

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 6 апреля 2018 г. в 22:13:22

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Khushboo Vashi <khushboo.vashi@enterprisedb.com> 9 апреля 2018 г. в 08:59:38

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 10 апреля 2018 г. в 19:00:05

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 18 апреля 2018 г. в 17:22:58

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 24 апреля 2018 г. в 22:13:37

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 25 апреля 2018 г. в 10:52:34

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 27 апреля 2018 г. в 12:15:38

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 27 апреля 2018 г. в 12:34:36

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 28 апреля 2018 г. в 01:25:41

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 18:00:49

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 18:03:03

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 30 апреля 2018 г. в 18:28:17

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 18:33:17

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 30 апреля 2018 г. в 18:37:01

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 18:45:21

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 30 апреля 2018 г. в 20:57:56

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 2 мая 2018 г. в 08:30:11

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 2 мая 2018 г. в 11:08:00

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 2 мая 2018 г. в 17:54:16

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 5 мая 2018 г. в 01:01:14

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 5 мая 2018 г. в 01:03:08

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 7 мая 2018 г. в 22:01:08

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Khushboo Vashi <khushboo.vashi@enterprisedb.com> 9 мая 2018 г. в 14:02:56

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Akshay Joshi <akshay.joshi@enterprisedb.com> 9 мая 2018 г. в 15:38:53

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 9 мая 2018 г. в 18:00:01

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 10 мая 2018 г. в 09:52:29

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 10 мая 2018 г. в 17:38:31

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 11 мая 2018 г. в 09:31:53

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 12 мая 2018 г. в 00:27:25

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 12 мая 2018 г. в 02:10:49

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 14 мая 2018 г. в 12:29:56

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 14 мая 2018 г. в 15:38:46

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 14 мая 2018 г. в 15:40:16

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 14 мая 2018 г. в 15:41:44

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 15 мая 2018 г. в 13:37:08

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 15 мая 2018 г. в 18:03:56

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 16 мая 2018 г. в 10:20:10

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 16 мая 2018 г. в 18:25:40

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 24 мая 2018 г. в 09:59:16

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 24 мая 2018 г. в 17:43:02

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Robert Eckhardt <reckhardt@pivotal.io> 31 мая 2018 г. в 17:39:05

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 31 мая 2018 г. в 18:13:18

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 1 июня 2018 г. в 14:52:33

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 1 июня 2018 г. в 18:17:45

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Victoria Henry <vhenry@pivotal.io> 1 июня 2018 г. в 19:39:01

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 1 июня 2018 г. в 21:36:26

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Victoria Henry <vhenry@pivotal.io> 1 июня 2018 г. в 22:17:23

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> 6 июня 2018 г. в 14:07:50

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Victoria Henry <vhenry@pivotal.io> 6 июня 2018 г. в 18:25:13

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> 7 июня 2018 г. в 07:35:16

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Victoria Henry <vhenry@pivotal.io> 7 июня 2018 г. в 18:21:52

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> 11 июня 2018 г. в 08:44:44

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 21 мая 2018 г. в 16:46:02

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 21 мая 2018 г. в 16:56:30

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 23 мая 2018 г. в 17:14:40

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Dave Page <dpage@pgadmin.org> 23 мая 2018 г. в 17:16:35

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Khushboo Vashi <khushboo.vashi@enterprisedb.com> 7 июня 2018 г. в 10:41:33

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 11 июня 2018 г. в 13:29:22

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 11 июня 2018 г. в 13:35:52

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 09:16:47

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Anthony Emengo <aemengo@pivotal.io> 30 апреля 2018 г. в 13:21:08

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Ashesh Vashi <ashesh.vashi@enterprisedb.com> 30 апреля 2018 г. в 13:24:58

Re: [pgadmin4][patch] Initial patch to decouple from ACI Tree Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> 30 апреля 2018 г. в 17:57:21

On Thu, May 10, 2018 at 8:08 PM, Anthony Emengo <aemengo@pivotal.io> wrote:

1. In TreeNode, we're keepging the reference of DOMElement, do we really need it?
As of right now, our Tree abstraction acts as an adapter to the aciTree library. The aciTree library needs the domElement for most of its functions (setInode, unload, etc). Thus this is the easiest way to introduce our abstraction and keep the functionality as before - at least until we decide that whether we want to switch out the library or not.

I understand that. But - I've not seen any reference of domElement the code yet, hence - pointed that out.

2. Are you expecting the tree class to be a singleton class
Since this tree is referenced throughout the codebase, considering it to be a singleton seems like the most appropriate pattern for this usecase. It is very much the same way how we create a single instance of the aciTree library and use that throughout the codebase. Moreover, it opens up opportunities to improve performance, for example caching lockups of nodes. I’m not a fan of singletons myself, but I feel like we’re simply keeping the architecture the same in the instance.

Yeah - I don't see any usage of tree object from anywhere.

And, we're already creating new object in browser.js (and, not utitlizing that instance anywhere.)

-- Thanks, Ashesh

Sincerely,

Anthony and Victoria

В списке pgadmin-hackers по дате отправления

Предыдущее

От: Aditya Toshniwal

Дата: 11 мая 2018 г. в 09:28:50

Сообщение: Re: JS Test error on Windows

Следующее

От: Dave Page

Дата: 11 мая 2018 г. в 11:33:15

Сообщение: pgAdmin 4 commit: Ensure server cleanup on exit only happens if theser