On 2021-Apr-07, Jehan-Guillaume de Rorthais wrote:
> When I'm creating a new node, I'm using the "pgaTester" factory class. It
> relies on PATH to check the major version using pg_config, then loads the
> appropriate class.
From a code cleanliness point of view, I agree that having separate
classes for each version is neater than what you call a forest of
conditionals. I'm not sure I like the way you instantiate the classes
in pgaTester though -- wouldn't it be saner to have PostgresNode::new
itself be in charge of deciding which class to bless the object as?
Since we're talking about modifying PostgresNode itself in order to
support this, it would make sense to do that.
(I understand that one of your decisions was to avoid modifying
PostgresNode, so that you could ingest whatever came out of PGDG without
having to patch it each time.)
--
Álvaro Herrera Valdivia, Chile