Can you get this polished off on Monday please Ashesh? On Thu, May 4, 2017 at 12:51 PM, Robert Eckhardt <reckha...@pivotal.io> wrote:
> All, > > This change in the xss testing is preventing our CI from going green and > is also preventing the Dependents and Dependencies tabs in Greenplum from > being useful. > > Can we either merge this or provide feedback as to what needs to change so > that it can be merged. > > Thank you > Rob > > On Tue, May 2, 2017 at 5:17 PM, Sarah McAlear <smcal...@pivotal.io> wrote: > >> Hi Hackers & Ashesh! >> >> Is there anything else we can do for this? >> >> Thanks! >> Matt & Sarah >> >> On Thu, Apr 27, 2017 at 10:37 AM, Joao Pedro De Almeida Pereira < >> jdealmeidapere...@pivotal.io> wrote: >> >>> Thanks for reviewing, Ashesh. >>> >>> We have updated the patch. The headers are all consistent and we removed >>> the __init__.py files in directories containing only .sql. >>> >>> Thanks! >>> Joao & Matt >>> >>> On Wed, Apr 26, 2017 at 11:22 AM, Joao Pedro De Almeida Pereira < >>> jdealmeidapere...@pivotal.io> wrote: >>> >>>> Hello Ashesh, >>>> >>>> Thanks for reviewing the patch. >>>> >>>> We added the __init__.py files into templates to convert them into >>>> packages so that the tests inside of them can be found by the test runner. >>>> >>>> Thanks! >>>> Joao & Sarah >>>> >>>> On Wed, Apr 26, 2017 at 1:26 AM, Ashesh Vashi < >>>> ashesh.va...@enterprisedb.com> wrote: >>>> >>>>> On Mon, Apr 24, 2017 at 4:43 PM, Dave Page <dp...@pgadmin.org> wrote: >>>>> >>>>>> Ashesh, can you review/commit this please? >>>>>> >>>>>> On Fri, Apr 21, 2017 at 8:42 PM, Joao Pedro De Almeida Pereira < >>>>>> jdealmeidapere...@pivotal.io> wrote: >>>>>> >>>>>>> Hi Hackers, >>>>>>> >>>>>>> We found out that when you are connected to a GreenPlum database and >>>>>>> try to get Dependents and Dependencies of an object the application was >>>>>>> returning a SQL error. >>>>>>> >>>>>>> This patch splits the SQL query used to retrieve the Dependents, >>>>>>> Dependencies, and Roles SQL file into multiple versioned files. >>>>>>> Add Unit Tests for each file. >>>>>>> Also added __init__.py files to other test directories to run the >>>>>>> tests in them. >>>>>>> >>>>>> Hi Joao & Sarah, >>>>> >>>>> Why do we need to add __init__.py in the template directory? >>>>> I didn't understand the purpose of the adding __init__.py files in the >>>>> template directories. >>>>> >>>>> NOTE: The headers in those files are not consistent with the other >>>>> project files. >>>>> >>>>> -- >>>>> >>>>> Thanks & Regards, >>>>> >>>>> Ashesh Vashi >>>>> EnterpriseDB INDIA: Enterprise PostgreSQL Company >>>>> <http://www.enterprisedb.com/> >>>>> >>>>> >>>>> *http://www.linkedin.com/in/asheshvashi* >>>>> <http://www.linkedin.com/in/asheshvashi> >>>>> >>>>>> Add ORDER BY into Copy Selection Feature test to ensure the results >>>>>>> are retrieved always in the same order >>>>>>> Renamed the Scenario of the xss_checks_pgadmin_debugger_test and >>>>>>> skip it for versions less than 9.1 >>>>>>> >>>>>>> Thanks >>>>>>> >>>>>>> Joao & Sarah >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> Sent via pgadmin-hackers mailing list (pgadmin-hack...@postgresql.or >>>>>>> g) >>>>>>> To make changes to your subscription: >>>>>>> http://www.postgresql.org/mailpref/pgadmin-hackers >>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> Dave Page >>>>>> Blog: http://pgsnake.blogspot.com >>>>>> Twitter: @pgsnake >>>>>> >>>>>> EnterpriseDB UK: http://www.enterprisedb.com >>>>>> The Enterprise PostgreSQL Company >>>>>> >>>>> >>>>> >>>> >>> >>> >>> -- >>> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org) >>> To make changes to your subscription: >>> http://www.postgresql.org/mailpref/pgadmin-hackers >>> >>> >> > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company