Merge lp:~stewart/drizzle/kill-HAS_DOES_TRANSACTIONS into lp:drizzle

Proposed by Stewart Smith
Status: Merged
Merged at revision: 2621
Proposed branch: lp:~stewart/drizzle/kill-HAS_DOES_TRANSACTIONS
Merge into: lp:drizzle
Diff against target: 60 lines (+3/-7)
4 files modified
drizzled/cursor.cc (+1/-1)
drizzled/plugin/storage_engine.h (+0/-2)
plugin/innobase/handler/ha_innodb.cc (+1/-2)
plugin/storage_engine_api_tester/storage_engine_api_tester.cc (+1/-2)
To merge this branch: bzr merge lp:~stewart/drizzle/kill-HAS_DOES_TRANSACTIONS
Reviewer Review Type Date Requested Status
Stewart Smith (community) Approve
Drizzle Trunk Pending
Review via email: mp+149947@code.launchpad.net

Description of the change

We don't need this HTON bit anymore, we can interrogate the actual class.

To post a comment you must log in.
Revision history for this message
Stewart Smith (stewart) wrote :

Brian acked via private mail.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'drizzled/cursor.cc'
--- drizzled/cursor.cc 2013-01-29 13:10:03 +0000
+++ drizzled/cursor.cc 2013-02-21 22:37:21 +0000
@@ -175,7 +175,7 @@
175175
176bool Cursor::has_transactions()176bool Cursor::has_transactions()
177{177{
178 return (getTable()->getShare()->db_type()->check_flag(HTON_BIT_DOES_TRANSACTIONS));178 return (getEngine()->participatesInSqlTransaction());
179}179}
180180
181void Cursor::ha_statistic_increment(uint64_t system_status_var::*offset) const181void Cursor::ha_statistic_increment(uint64_t system_status_var::*offset) const
182182
=== modified file 'drizzled/plugin/storage_engine.h'
--- drizzled/plugin/storage_engine.h 2011-07-15 08:26:31 +0000
+++ drizzled/plugin/storage_engine.h 2013-02-21 22:37:21 +0000
@@ -51,7 +51,6 @@
51 HTON_BIT_NOT_USER_SELECTABLE,51 HTON_BIT_NOT_USER_SELECTABLE,
52 HTON_BIT_TEMPORARY_NOT_SUPPORTED, // Having temporary tables not supported52 HTON_BIT_TEMPORARY_NOT_SUPPORTED, // Having temporary tables not supported
53 HTON_BIT_TEMPORARY_ONLY,53 HTON_BIT_TEMPORARY_ONLY,
54 HTON_BIT_DOES_TRANSACTIONS,
55 HTON_BIT_STATS_RECORDS_IS_EXACT,54 HTON_BIT_STATS_RECORDS_IS_EXACT,
56 HTON_BIT_NULL_IN_KEY,55 HTON_BIT_NULL_IN_KEY,
57 HTON_BIT_CAN_INDEX_BLOBS,56 HTON_BIT_CAN_INDEX_BLOBS,
@@ -81,7 +80,6 @@
81static const std::bitset<HTON_BIT_SIZE> HTON_NOT_USER_SELECTABLE(1 << HTON_BIT_NOT_USER_SELECTABLE);80static const std::bitset<HTON_BIT_SIZE> HTON_NOT_USER_SELECTABLE(1 << HTON_BIT_NOT_USER_SELECTABLE);
82static const std::bitset<HTON_BIT_SIZE> HTON_TEMPORARY_NOT_SUPPORTED(1 << HTON_BIT_TEMPORARY_NOT_SUPPORTED);81static const std::bitset<HTON_BIT_SIZE> HTON_TEMPORARY_NOT_SUPPORTED(1 << HTON_BIT_TEMPORARY_NOT_SUPPORTED);
83static const std::bitset<HTON_BIT_SIZE> HTON_TEMPORARY_ONLY(1 << HTON_BIT_TEMPORARY_ONLY);82static const std::bitset<HTON_BIT_SIZE> HTON_TEMPORARY_ONLY(1 << HTON_BIT_TEMPORARY_ONLY);
84static const std::bitset<HTON_BIT_SIZE> HTON_HAS_DOES_TRANSACTIONS(1 << HTON_BIT_DOES_TRANSACTIONS);
85static const std::bitset<HTON_BIT_SIZE> HTON_STATS_RECORDS_IS_EXACT(1 << HTON_BIT_STATS_RECORDS_IS_EXACT);83static const std::bitset<HTON_BIT_SIZE> HTON_STATS_RECORDS_IS_EXACT(1 << HTON_BIT_STATS_RECORDS_IS_EXACT);
86static const std::bitset<HTON_BIT_SIZE> HTON_NULL_IN_KEY(1 << HTON_BIT_NULL_IN_KEY);84static const std::bitset<HTON_BIT_SIZE> HTON_NULL_IN_KEY(1 << HTON_BIT_NULL_IN_KEY);
87static const std::bitset<HTON_BIT_SIZE> HTON_CAN_INDEX_BLOBS(1 << HTON_BIT_CAN_INDEX_BLOBS);85static const std::bitset<HTON_BIT_SIZE> HTON_CAN_INDEX_BLOBS(1 << HTON_BIT_CAN_INDEX_BLOBS);
8886
=== modified file 'plugin/innobase/handler/ha_innodb.cc'
--- plugin/innobase/handler/ha_innodb.cc 2012-10-20 23:08:23 +0000
+++ plugin/innobase/handler/ha_innodb.cc 2013-02-21 22:37:21 +0000
@@ -395,8 +395,7 @@
395 HTON_PRIMARY_KEY_IN_READ_INDEX |395 HTON_PRIMARY_KEY_IN_READ_INDEX |
396 HTON_PARTIAL_COLUMN_READ |396 HTON_PARTIAL_COLUMN_READ |
397 HTON_TABLE_SCAN_ON_INDEX |397 HTON_TABLE_SCAN_ON_INDEX |
398 HTON_HAS_FOREIGN_KEYS |398 HTON_HAS_FOREIGN_KEYS)
399 HTON_HAS_DOES_TRANSACTIONS)
400 {399 {
401 table_definition_ext= plugin::DEFAULT_DEFINITION_FILE_EXT;400 table_definition_ext= plugin::DEFAULT_DEFINITION_FILE_EXT;
402 addAlias("INNOBASE");401 addAlias("INNOBASE");
403402
=== modified file 'plugin/storage_engine_api_tester/storage_engine_api_tester.cc'
--- plugin/storage_engine_api_tester/storage_engine_api_tester.cc 2012-01-15 20:54:59 +0000
+++ plugin/storage_engine_api_tester/storage_engine_api_tester.cc 2013-02-21 22:37:21 +0000
@@ -492,8 +492,7 @@
492 HTON_PRIMARY_KEY_IN_READ_INDEX |492 HTON_PRIMARY_KEY_IN_READ_INDEX |
493 HTON_PARTIAL_COLUMN_READ |493 HTON_PARTIAL_COLUMN_READ |
494 HTON_TABLE_SCAN_ON_INDEX |494 HTON_TABLE_SCAN_ON_INDEX |
495 HTON_HAS_FOREIGN_KEYS |495 HTON_HAS_FOREIGN_KEYS)
496 HTON_HAS_DOES_TRANSACTIONS)
497 {496 {
498 ENGINE_NEW_STATE("::SEAPITester()");497 ENGINE_NEW_STATE("::SEAPITester()");
499 }498 }

Subscribers

People subscribed via source and target branches

to all changes: