Added command execution time measurement

[MakSl]

This will be useful to keep track of possible performance degradation when code changes.

[demonolock]

Думаю ничего страшного, если будем замерять время команд всегда

[MakSl]

Хмм, получается будем замерять всегда, но будем использовать не всегда?
Может лучше оставим условие? Хоть замер и не занимает много времени. Но читаемость кода так лучше имхо.

[demonolock]

Почему читаемость лучше?

 if self.execution_time:
                start_time = time.time()
                self.test_class.output = subprocess.check_output(
                    cmdline,
                    stderr=subprocess.STDOUT,
                    env=env
                ).decode('utf-8', errors='replace')
                end_time = time.time()
                self.execution_time = end_time - start_time
            else:
                self.test_class.output = subprocess.check_output(
                    cmdline,
                    stderr=subprocess.STDOUT,
                    env=env
                ).decode('utf-8', errors='replace')

против

                start_time = time.time()
                self.test_class.output = subprocess.check_output(
                    cmdline,
                    stderr=subprocess.STDOUT,
                    env=env
                ).decode('utf-8', errors='replace')
                end_time = time.time()
                self.execution_time = end_time - start_time      

[MakSl]

В конечной реализации это доп условие и вправду лишнее.

[demonolock]

прокидывать сюда не нужно было, execution_time и так доступен из self.execution_time

[MakSl]

Поправил

[demonolock]

Проблема со стилем кода

• flake8 .
  ./build/lib/testgres/performance_test.py:3:1: E302 expected 2 blank lines, found 1
  ./testgres/performance_test.py:3:1: E302 expected 2 blank lines, found 1
  https://app.travis-ci.com/github/postgrespro/testgres/jobs/622019828

Можно проверять локально поставив flake8
Установка
pip install flake8

Проверка
flake8 /путь/до_кода

[demonolock]

А тест проверял? Проходит?
Не вижу check_time в функции backup_node

[MakSl]

Данный тест был не работоспособен. Исправил его и проверил у себя. Получение времени осуществляется.

Отдельный тест на проверку работоспособности считаю излишней. Поэтому по итогу убрал тест.

[demonolock]

Так лучше не делать
execution_time и признак того что мы хотим замерить время и само время.
Python конечно позволяет, но это ведет к сложноотслеживаемым ошибкам. Лучше сделать его None если замера не было

[MakSl]

Согласен. Исправил этот момент.