Skip to content
This repository was archived by the owner on Jan 27, 2021. It is now read-only.

Implement read and create benchmark in Nextras ORM#26

Merged
sergeyklay merged 5 commits into
sergeyklay:masterfrom
repli2dev:nextras-orm
Feb 19, 2018
Merged

Implement read and create benchmark in Nextras ORM#26
sergeyklay merged 5 commits into
sergeyklay:masterfrom
repli2dev:nextras-orm

Conversation

@repli2dev
Copy link
Copy Markdown

@repli2dev repli2dev commented Feb 14, 2018

Hi, I have implemented the tests create and read in Nextras ORM (#23).

@sergeyklay
Copy link
Copy Markdown
Owner

sergeyklay commented Feb 14, 2018

@repli2dev Could you please README.md? You'll need to amend table with results and build id from the Travis CI

@repli2dev
Copy link
Copy Markdown
Author

@sergeyklay I will do, however I am having some troubles making it work on Travis CI as I am getting: Nextras\Dbal\QueryException: Unknown or incorrect time zone: 'UTC' on /home/travis/build/sergeyklay/php-orm-benchmark/provider/nextrasorm/vendor/nextras/dbal/src/Drivers/Mysqli/MysqliDriver.php:381 which typically means that I need to add timezones to the MySQL and I don't know the prefered way.

@sergeyklay
Copy link
Copy Markdown
Owner

@repli2dev

The known approach to fix it is add this command to before_script section:

mysql_tzinfo_to_sql /usr/share/zoneinfo | mysql -u root mysql

Have a nice day

@repli2dev
Copy link
Copy Markdown
Author

@sergeyklay Now it has executed properly, however I am a bit puzzled which results are which in the CI to update the README.

@sergeyklay sergeyklay merged commit e318aec into sergeyklay:master Feb 19, 2018
@sergeyklay
Copy link
Copy Markdown
Owner

sergeyklay commented Feb 19, 2018

@repli2dev Let me update results a bit later. Thank you for the contributing, and for helping make benchmark better.

@repli2dev
Copy link
Copy Markdown
Author

Thanks... Ping me when other operations will need implementation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants