|From:||Nikolay Shaplov <dhyan(at)nataraj(dot)su>|
|Cc:||Michael Paquier <michael(at)paquier(dot)xyz>, Adam Berlin <berlin(dot)ab(at)gmail(dot)com>|
|Subject:||Re: Add extension options to control TAP and isolation tests|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
В письме от 10 ноября 2018 09:14:19 пользователь Michael Paquier написал:
> > Nice cleanup. Also, I like the ability to enable/control more types of
> > tests easily from the Makefile. What are the next steps for this
> > patch?
> Thanks. It seems to me that a complete review is still in order,
> particularly regarding the new makefile option names. And then, if
> everybody caring about the topic is happy with the way the patch is
> shaped, it can be carried over to being committed, which would be most
> likely something I'll do.
Is it ok, if I join the reviewing? I like test, especially TAP one, you know
Since you are much more experienced in postgres then me, I'd try to understand
how does the patch work, try to use if for writing more TAP test, and will
report problems and thoughts I came across while doing that.
So far while first reading the patch I came to following two
@@ -1,3 +1,3 @@
# Generated subdirectories
For me name "output_iso" means nothing. iso is something about CD/DVD or about
standards. I would not guess that iso stands for isolation if I did not know
it already. isolation_output is more sensible: I have heard that there are
some isolation tests, this must be something about it. May be it would be
better to change it to isolation_output everywhere instead of changing to
@@ -1293,6 +1293,34 @@ include $(PGXS)
+ list of isolation test cases
+ additional switches to pass to
+ switch defining if TAP tests need to be run
I tried to find definition in documentation what does "isolation test" exactly
means, but did not find. There is some general words about TAP tests in main
but I would not understand anything from it if I did not already know how it
In current extend-pgxs documentation there is some explanation about
regression test, it sensible enough. Since TAP and isolation tests are
introduced now, there should be same short explanation for both of them.
And also it would be good to add links from extend-pgxs to regress-tap and
regress saying that for more info about these tests one can look at postgres
doc, because they work in a similar way.
That's all so far. I'll look more into it later...
Do code for fun.
|Next Message||Magnus Hagander||2018-11-20 16:41:33||Re: Connection slots reserved for replication|
|Previous Message||Stephen Frost||2018-11-20 16:25:22||Re: [RFC] Removing "magic" oids|