hivemeta 0.0.4 → 0.0.5

data/CHANGELOG

@@ -1,5 +1,9 @@
 * 2011-05-19 - fsf
-- perf: remove unnecessary string indexed assignment - thanks ruby-prof!
+- thank you ruby-prof!
+- perf: 4x+ faster ... now basically on par with manual split into array
+- perf: create extra hash for column index by name
+- perf: remove unnecessary string indexed assignment
+- clean: table.each does each inside rather than each_with_index
 
 * 2011-05-17 - fsf
 - bugfix: default unspecified delimiter is ^A rather than TAB

data/lib/hivemeta/connection.rb

@@ -77,8 +77,9 @@ module HiveMeta
           col_cmt  = rec[2]
           tbl_loc  = rec[3]
           sd_id    = rec[4]
-          table.columns[col_idx]  = col_name
-          table.comments[col_idx] = col_cmt
+          table.columns[col_idx]         = col_name
+          table.indexes[col_name.to_sym] = col_idx
+          table.comments[col_idx]        = col_cmt
           table.path      = tbl_loc
         end
 

data/lib/hivemeta/record.rb

@@ -9,10 +9,11 @@ module HiveMeta
         raise FieldCountError
       end
 
-      @columns = {}
-      table.each_col_with_index do |col_name, i|
-        @columns[col_name.to_sym] = @fields[i]
-      end
+      @table = table
+      #@columns = {}
+#      table.each_col_with_index do |col_name, i|
+#        #@columns[col_name.to_sym] = @fields[i]
+#      end
     end
 
     # allow for column access via column name as an index
@@ -22,13 +23,15 @@ module HiveMeta
     # example: rec[7]
     def [] index
       return "#{@fields[index]}" if index.is_a? Integer
-      "#{@columns[index.to_sym]}"
+      #"#{@columns[index.to_sym]}"
+      "#{@fields[@table.indexes[index.to_sym]]}"
     end
 
     # allow for column access via column name as a method
     # example: rec.col_name
     def method_missing(id, *args)
-      return @columns[id] if @columns[id]
+      return @fields[@table.indexes[id]] if @fields[@table.indexes[id]]
+      #return @columns[id] if @columns[id]
       raise NoMethodError
     end
   end

data/lib/hivemeta/table.rb

@@ -4,12 +4,13 @@ module HiveMeta
     include Comparable
     include Enumerable
 
-    attr_accessor :path, :columns, :comments, :delimiter
+    attr_accessor :path, :columns, :comments, :delimiter, :indexes
 
     def initialize(name)
       @name = name
       @path = nil
-      @columns   = []
+      @indexes   = {} # column indexes by name
+      @columns   = [] # column names by index
       @comments  = []
       @delimiter = "\001"
     end
@@ -19,7 +20,7 @@ module HiveMeta
     end
 
     def each
-      @columns.each_with_index do |column_name, index|
+      @columns.each do |column_name|
         yield column_name if column_name
       end
     end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 0
-  - 4
-  version: 0.0.4
+  - 5
+  version: 0.0.5
 platform: ruby
 authors: 
 - Frank Fejes