Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Insert slonik preamble and execute script instead of having user do it #90

Merged
merged 9 commits into from
Jul 7, 2015

Conversation

nkiraly
Copy link
Member

@nkiraly nkiraly commented Jul 7, 2015

For #89 , fix slonik preamble, node storage, and subscribe statement slonik output files to work without modification.

…nify the node create and subscription steps as whole pieces
e.g.
Fatal error: Uncaught exception 'LogicException' with message 'Missing stream url, the stream can not be opened. This may be caused by a premature call to close().' in /home/nicholas.kiraly/engineering/DBSteward/vendor/monolog/monolog/src/Monolog/Handler/StreamHandler.php:73
Stack trace:
#0 /home/nicholas.kiraly/engineering/DBSteward/vendor/monolog/monolog/src/Monolog/Handler/AbstractProcessingHandler.php(37): Monolog\Handler\StreamHandler->write(Array)
#1 /home/nicholas.kiraly/engineering/DBSteward/vendor/monolog/monolog/src/Monolog/Logger.php(269): Monolog\Handler\AbstractProcessingHandler->handle(Array)
#2 /home/nicholas.kiraly/engineering/DBSteward/vendor/monolog/monolog/src/Monolog/Logger.php(447): Monolog\Logger->addRecord(250, '[File Segment] ...', Array)
#3 /home/nicholas.kiraly/engineering/DBSteward/lib/DBSteward/dbsteward.php(936): Monolog\Logger->log(250, '[File Segment] ...')
#4 /home/nicholas.kiraly/engineering/DBSteward/lib/DBSteward/dbsteward.php(950): dbsteward::log(250, '[File Segment] ...')
#5 /home/nich in /home/nicholas.kiraly/engineering/DBSteward/vendor/monolog/monolog/src/Monolog/Handler/StreamHandler.php on line 73
… 1 and 3 .slonik files

TODO: need to collect sql_stage1_file names from diff_doc() if there is
more than one as a result of many changes between definition files
nkiraly added a commit that referenced this pull request Jul 7, 2015
…cript

Insert slonik preamble and execute script instead of having user do it
@nkiraly nkiraly merged commit a89bd81 into master Jul 7, 2015
@nkiraly nkiraly deleted the feature/slonik-preamble-execute-script branch July 7, 2015 05:44
nkiraly added a commit that referenced this pull request Jul 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant