← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/heap/hp_close.cc

  • Committer: Prafulla Tekawade
  • Date: 2010-07-18 03:36:32 UTC
  • mto: (1662.1.4 rollup)
  • mto: This revision was merged to the branch mainline in revision 1664.
  • Revision ID: prafulla_t@users.sourceforge.net-20100718033632-p7q6qtgliqbhe38p
Fix for Bug 592444

There were two problems:
o. In greedy_search optimizer method, best_extension_by_limited search
   maintains join embedding(nestedness) of tables added so far, so that 
   correct(valid)  join order is selected
   These are requirements from nested outer join executioner.
   The problem was, embedding_map was not correctly updated when a table 
   is added to optimal plan outside best_extension_by_limited search, 
   by greedy_search method. We need to update join->cur_embedding_map
   correctly here so that execution plan for other tables get
   generated.
   Invoked checked_interleaving_with_nj from greedy_search on the
   best_table selected. Fixed its prototype to take only one JoinTab
   This is same as mysql 5.1 source tree.
o. The other problem was, join->cur_embedding_map was not restored correctly
   when a table is added to the optimal plan to reflect the current embedding 
   map. 
   Taken good documented method restore_prev_nj_state which restores 
   cur_embedding_map from mysql 5.1 source tree and modified it for drizzled 
   code.

Show diffs side-by-side

added added

removed removed

Lines of Context:
plugin/heap/hp_close.cc
11
11
 
12
12
   You should have received a copy of the GNU General Public License
13
13
   along with this program; if not, write to the Free Software
14
 
   Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA */
 
14
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA */
15
15
 
16
16
/* close a heap-database */
17
17
 
26
26
int heap_close(HP_INFO *info)
27
27
{
28
28
  int tmp;
29
 
  THR_LOCK_heap.lock();
 
29
  pthread_mutex_lock(&THR_LOCK_heap);
30
30
  tmp= hp_close(info);
31
 
  THR_LOCK_heap.unlock();
32
 
 
 
31
  pthread_mutex_unlock(&THR_LOCK_heap);
33
32
  return(tmp);
34
33
}
35
34
 
37
36
int hp_close(HP_INFO *info)
38
37
{
39
38
  int error=0;
40
 
  info->getShare()->changed=0;
 
39
  info->s->changed=0;
41
40
  heap_open_list.remove(info);
42
 
  if (!--info->getShare()->open_count && info->getShare()->delete_on_close)
43
 
    hp_free(info->getShare());                          /* Table was deleted */
44
 
  delete info;
 
41
  if (!--info->s->open_count && info->s->delete_on_close)
 
42
    hp_free(info->s);                           /* Table was deleted */
 
43
  free((unsigned char*) info);
45
44
  return(error);
46
45
}