public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Priyanka Shendge <[email protected]>
Cc: Navnath Gadakh <[email protected]>
Cc: Kanchan Mohitey <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Server specific advanced config file for testsuite
Date: Wed, 14 Sep 2016 13:13:16 +0100
Message-ID: <CA+OCxoxseRbGUwbvf_r_Sy5sDXm+YY=PRdUq_EQf7WcpzLYHFQ@mail.gmail.com> (raw)
In-Reply-To: <CAKmZXFQxpOPrkRfQxy+0y9uad3=baQR2-CmxtVR_P7fh4jRtVA@mail.gmail.com>
References: <CAOAJCYpsPXH3oKen-zH3oLjuOqdrt=XE4FiazYm0MsVQWtXk9g@mail.gmail.com>
<CA+OCxox=16k4tAhdZLDk_D0bqSG34=rd4QfXgcqGx=61GjL-uA@mail.gmail.com>
<CAKmZXFQxpOPrkRfQxy+0y9uad3=baQR2-CmxtVR_P7fh4jRtVA@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
Hi
On Tue, Sep 13, 2016 at 1:09 PM, Priyanka Shendge
<[email protected]> wrote:
>
>
> On 13 September 2016 at 14:18, Dave Page <[email protected]> wrote:
>>
>> Hi
>>
>> On Mon, Sep 12, 2016 at 6:26 PM, Navnath Gadakh
>> <[email protected]> wrote:
>> > Hi Dave,
>> > As we know that, we are using two JSON files one is for basic
>> > configuration of server(test_config.json) and second for detailed
>> > configuration of server(test_advanced_config.json).
>> > In test_advanced_config.json, we give detail for database, schema,
>> > extension, collation etc. These details are generic I mean these details
>> > are
>> > not server specific.
>> > - Do we need to allow user to give server specific details?
>>
>> I don't see any reason to do so. Honestly I don't see much need for
>> test_advanced_config.json at all - I can't see anyone ever customising
>> the tests. I certainly doubt I will.
>
>
> As per your suggestion, I have added test data to test_advanced_config.json
> which will be used/changed by advanced user.
I suggested it because I didn't think it should all be in the main
config file. I didn't realise it was going to get so big at the time.
> test_advanced_config.json contain test data for all nodes (i.e. database,
> schema, tablespace etc..) except server.
> If we dont use test_advanced_config.json file, from where we can fetch test
> data for all nodes?
Hard-code it in the tests.
> We wants to edit test_advanced_config.json as per server type and version as
> there are 2 types of
> servers available i.e. PostgreSQL and EPAS which might have different test
> data as per server type.
> e.g. "owner": in case of PG it can be owner: "postgres"
> which may not be available for EPAS, for EPAS it can be "enterprisedb" or
> vice versa.
But that data is in the server config (test_config.json).
> If we do not modify the tests and run testsuite it will fail for multiple
> servers.
I honestly don't see why anything other than server-specific values
(i.e. connection info, tablespace path) needs to be customisable.
--
Dave Page
VP, Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
--
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers
view thread (7+ messages)
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], [email protected]
Subject: Re: Server specific advanced config file for testsuite
In-Reply-To: <CA+OCxoxseRbGUwbvf_r_Sy5sDXm+YY=PRdUq_EQf7WcpzLYHFQ@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