← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/math/atan.h

  • 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:
drizzled/function/math/atan.h
17
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
18
18
 */
19
19
 
20
 
#ifndef PLUGIN_MATH_FUNCTIONS_ATAN_H
21
 
#define PLUGIN_MATH_FUNCTIONS_ATAN_H
 
20
#ifndef DRIZZLED_FUNCTION_MATH_ATAN_H
 
21
#define DRIZZLED_FUNCTION_MATH_ATAN_H
22
22
 
23
23
#include <drizzled/function/func.h>
24
24
#include <drizzled/function/math/dec.h>
29
29
class Item_func_atan :public Item_dec_func
30
30
{
31
31
public:
32
 
  Item_func_atan() :Item_dec_func() {}
 
32
  Item_func_atan(Item *a) :Item_dec_func(a) {}
 
33
  Item_func_atan(Item *a,Item *b) :Item_dec_func(a,b) {}
33
34
  double val_real();
34
35
  const char *func_name() const { return "atan"; }
35
 
  bool check_argument_count(int n) { return n == 1 || n == 2; }
36
36
};
37
37
 
38
38
} /* namespace drizzled */
39
39
 
40
 
#endif /* PLUGIN_MATH_FUNCTIONS_ATAN_H */
 
40
#endif /* DRIZZLED_FUNCTION_MATH_ATAN_H */