maria-developers team mailing list archive

[Sergey Petrunya <psergey@xxxxxxxxxxxx>]

At file:///home/psergey/dev/maria-5.1-table-elim-r10/

------------------------------------------------------------
revno: 2724
revision-id: psergey@xxxxxxxxxxxx-20090815153912-q47vfp1j22ilmup2
parent: psergey@xxxxxxxxxxxx-20090815121442-706m9ujn8km4u4y1
committer: Sergey Petrunya <psergey@xxxxxxxxxxxx>
branch nick: maria-5.1-table-elim-r10
timestamp: Sat 2009-08-15 18:39:12 +0300
message:
  MWL#17: Table elimination
  - Review feedback, more variable renames 
=== modified file 'sql/opt_table_elimination.cc'
--- a/sql/opt_table_elimination.cc	2009-08-15 12:14:42 +0000
+++ b/sql/opt_table_elimination.cc	2009-08-15 15:39:12 +0000
@@ -114,7 +114,6 @@
     MODULE_EXPRESSION,
     MODULE_MULTI_EQUALITY,
     MODULE_UNIQUE_KEY,
-    MODULE_TABLE,
     MODULE_OUTER_JOIN
   } type; /* Type of the object */
   
@@ -138,7 +137,7 @@
 {
 public:
   Field_value *field;
-  Item  *val;
+  Item  *expression;
   
   /* Used during condition analysis only, similar to KEYUSE::level */
   uint level;
@@ -510,18 +509,18 @@
       */
       if (old->field == new_fields->field)
       {
-	if (!new_fields->val->const_item())
+	if (!new_fields->expression->const_item())
 	{
 	  /*
 	    If the value matches, we can use the key reference.
 	    If not, we keep it until we have examined all new values
 	  */
-	  if (old->val->eq(new_fields->val, old->field->field->binary()))
+	  if (old->expression->eq(new_fields->expression, old->field->field->binary()))
 	  {
 	    old->level= and_level;
 	  }
 	}
-	else if (old->val->eq_by_collation(new_fields->val, 
+	else if (old->expression->eq_by_collation(new_fields->expression, 
                                            old->field->field->binary(),
                                            old->field->field->charset()))
 	{
@@ -633,7 +632,7 @@
   /* Store possible eq field */
   (*eq_dep)->type=  Module_dep::MODULE_EXPRESSION; //psergey-todo;
   (*eq_dep)->field= get_field_value(te, field); 
-  (*eq_dep)->val=   *value;
+  (*eq_dep)->expression=   *value;
   (*eq_dep)->level= and_level;
   (*eq_dep)++;
 }
@@ -953,7 +952,7 @@
   {
     deps_setter.expr_offset= eq_dep - te->equality_deps;
     eq_dep->unknown_args= 0;
-    eq_dep->val->walk(&Item::check_column_usage_processor, FALSE, 
+    eq_dep->expression->walk(&Item::check_column_usage_processor, FALSE, 
                       (uchar*)&deps_setter);
     if (!eq_dep->unknown_args)
     {
@@ -1283,7 +1282,7 @@
     char buf[128];
     String str(buf, sizeof(buf), &my_charset_bin);
     str.length(0);
-    eq_dep->val->print(&str, QT_ORDINARY);
+    eq_dep->expression->print(&str, QT_ORDINARY);
     fprintf(DBUG_FILE, "  equality%d: %s -> %s.%s\n", 
             eq_dep - te->equality_deps,
             str.c_ptr(),