Merge lp:~vjsamuel/drizzle/rplugin-drizzle-protocol into lp:~drizzle-trunk/drizzle/development
Proposed by
Vijay Samuel
Status: | Superseded |
---|---|
Proposed branch: | lp:~vjsamuel/drizzle/rplugin-drizzle-protocol |
Merge into: | lp:~drizzle-trunk/drizzle/development |
Diff against target: |
558 lines (+165/-58) 29 files modified
drizzled/module/option_context.cc (+32/-3) drizzled/module/option_context.h (+3/-13) drizzled/module/option_map.h (+3/-8) drizzled/option.cc (+1/-1) drizzled/plugin.h (+2/-2) plugin/auth_file/auth_file.cc (+1/-1) plugin/auth_ldap/auth_ldap.cc (+1/-1) plugin/auth_ldap/test_ldap.sh (+1/-1) plugin/auth_test/auth_test.cc (+1/-1) plugin/blitzdb/ha_blitz.cc (+1/-1) plugin/crc32/crc32udf.cc (+1/-1) plugin/default_replicator/default_replicator.cc (+1/-1) plugin/drizzle_protocol/drizzle_protocol.cc (+99/-7) plugin/errmsg_notify/errmsg_notify.cc (+1/-1) plugin/filtered_replicator/filtered_replicator.cc (+1/-1) plugin/logging_stats/tests/t/command-master.opt (+1/-1) plugin/logging_stats/tests/t/max_session-master.opt (+1/-1) plugin/logging_stats/tests/t/max_user-master.opt (+1/-1) plugin/logging_stats/tests/t/slap-master.opt (+1/-1) plugin/rabbitmq/rabbitmq_log.cc (+1/-1) plugin/rand_function/rand_function.cc (+1/-1) plugin/replication_dictionary/module.cc (+1/-1) plugin/rot13/rot13.cc (+1/-1) plugin/simple_user_policy/module.cc (+1/-1) plugin/sleep/sleep.cc (+1/-1) plugin/syslog/module.cc (+1/-1) plugin/transaction_log/module.cc (+1/-1) plugin/version/versionudf.cc (+1/-1) tests/test-run.pl (+3/-2) |
To merge this branch: | bzr merge lp:~vjsamuel/drizzle/rplugin-drizzle-protocol |
Related bugs: | |
Related blueprints: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Monty Taylor | Needs Fixing | ||
Brian Aker | Pending | ||
Drizzle Developers | Pending | ||
Review via email: mp+28552@code.launchpad.net |
This proposal has been superseded by a proposal from 2010-07-06.
To post a comment you must log in.
Good work so far. Now that you're hacking in the server, there are a few more new things you're going to need to deal with. (Most of this has to do with error message handling)
First of all: in init(module: :Context &context), you're doing validation checks directly at module init time. I'd really prefer to see option value validation happen at option parse time, so that we don't have to get halfway in to server startup before we discover a bad value in a config file. Can you register validation methods for these values when you add them to the context?
Secondly, for reporting error messages, you need to use errmsg_printf rather than cout. This allows them to be processed by the servers error message reporting system, which may or may not be configured to actually print to a console (other options would be syslog or the like)
Just while I'm giving you feedback, if you _were_ to use a cout approach (not a choice here) you should be using cerr instead, and also should be adding a std::endl on to the end.
Finally, you need to wrap these new messages strings you just added in _() so that they'll get translated.
So this:
cout << "Invalid value for buffer_length";
Should become:
errmsg_ printf( ERRMSG_ LVL_ERROR, _("Invalid value for buffer_length\n"));