WL #2094 Federated Storage Handler

This is the first changeset of suggested changes recommended in Kostja's 
review of my patch, 1.1846, which includes only functionality changes. 
Style changes/Documentation patch to follow.


include/mysql.h:
  removed declaration of cli_fetch_lengths per Kostja's suggestion
libmysql/libmysql.c:
  moved mysql_fetch_lengths to client.c (for server to access) per Kostja's
  suggestion
sql-common/client.c:
  added back 'static' to function definition, added mysql_fetch_lengths
sql/ha_federated.cc:
  changed to use defines as opposed to hardcoded values
sql/ha_federated.h:
  took out duplicate table_flag, fixed a resolve mistake

2 files changed, 7 insertions, 11 deletions

diff --git a/sql/ha_federated.cc b/sql/ha_federated.cc
index ae6cb1f76be..cf704f9f569 100644
--- a/sql/ha_federated.cc
+++ b/sql/ha_federated.cc
@@ -560,10 +560,10 @@ static int parse_url(FEDERATED_SHARE *share, TABLE *table,
 
       if (!share->port)
       {
-        if (strcmp(share->hostname, "localhost") == 0)
-          share->socket= my_strdup("/tmp/mysql.sock", MYF(0));
+        if (strcmp(share->hostname, my_localhost) == 0)
+          share->socket= my_strdup(MYSQL_UNIX_ADDR, MYF(0));
         else
-          share->port= 3306;
+          share->port= MYSQL_PORT;
       }
 
       DBUG_PRINT("ha_federated::parse_url",
@@ -616,8 +616,7 @@ uint ha_federated::convert_row_to_internal_format(byte *record, MYSQL_ROW row)
   DBUG_ENTER("ha_federated::convert_row_to_internal_format");
 
   num_fields= mysql_num_fields(result);
-  lengths= (ulong*) my_malloc(num_fields * sizeof(ulong), MYF(0));
-  cli_fetch_lengths((ulong*) lengths, row, num_fields);
+  lengths= mysql_fetch_lengths(result);
 
   memset(record, 0, table->s->null_bytes);
 
@@ -628,8 +627,6 @@ uint ha_federated::convert_row_to_internal_format(byte *record, MYSQL_ROW row)
     else
       (*field)->store(row[x], lengths[x], &my_charset_bin);
   }
-  my_free((gptr) lengths, MYF(0));
-  lengths= 0;
 
   DBUG_RETURN(0);
 }
diff --git a/sql/ha_federated.h b/sql/ha_federated.h
index f685aa9e3f9..22fc03e9eec 100755
--- a/sql/ha_federated.h
+++ b/sql/ha_federated.h
@@ -76,7 +76,6 @@ private:
       return errorcode otherwise
   */
   uint convert_row_to_internal_format(byte *buf, MYSQL_ROW row);
-  bool create_where_from_key(String *to, KEY *key_info, const byte *key, uint key_length); 
   bool create_where_from_key(String *to, KEY *key_info, 
                              const byte *key, uint key_length); 
 
@@ -106,9 +105,9 @@ public:
   */
   ulong table_flags() const
   {
-    return (HA_TABLE_SCAN_ON_INDEX | HA_NOT_EXACT_COUNT | 
-            HA_PRIMARY_KEY_IN_READ_INDEX | HA_FILE_BASED | HA_AUTO_PART_KEY | 
-            HA_TABLE_SCAN_ON_INDEX | HA_CAN_INDEX_BLOBS);
+    return (HA_TABLE_SCAN_ON_INDEX | HA_NOT_EXACT_COUNT |
+            HA_PRIMARY_KEY_IN_READ_INDEX | HA_FILE_BASED |
+            HA_AUTO_PART_KEY | HA_CAN_INDEX_BLOBS);
   }
   /*
     This is a bitmap of flags that says how the storage engine