purview 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: caf3a73a4f73a939e96d27ecd75c682cb0c102d2
-  data.tar.gz: 1533be77323ba2b9b9acbaed18f0645320f0ee0a
+  metadata.gz: 8379bdb0d5f4798293f48485cb0a6f89808fb0a1
+  data.tar.gz: 2264aa219ba7d50a60b2797aef55e8a335260928
 SHA512:
-  metadata.gz: 72681a57612c06a773b8bf7fef5eb53dde36f660905c06107ebb0e21feac0edfe2ef09ee21c0193dc2b5ab5cb70af7f0ef8c61b489b7e6b2165b9f01d7dee602
-  data.tar.gz: 256c71e89cffa61cd77be9c465a94beef74408ec263bd1bf9c774665b5426040beaf2a73f3f5f1c4fdcec66dcb18cd042fe8f69dba87bf8f9828d2e341997ea0
+  metadata.gz: ce8958f9eb4556c2d3106d015fb8be223d1911e7cb8e6341e0151be5baede61410da4d098c7671579399ddecf7af6c7678b36f4a47f9d1296084738d4ef100ff
+  data.tar.gz: 5dc9a9313f0db29b31cba3b0700b09b86a819eeb4df70e59dc0eadedfd322de046c748930a9315e6ecedba0ea70d69d52327142de2cde64c732b00255d5ea964

data/CHANGELOG

@@ -1,6 +1,12 @@
+Release 1.0.1
+-------------
+
+* Use consistent naming for database connection-opts (this allowed some DRYing)
+* OPPREF: Only update `table_metadata` when creating/dropping tables
+
 Release 1.0.0
 -------------
 
-- Support on JRuby, MRI, and RBX for pulling, validating, parsing and loading
+* Support on JRuby, MRI, and RBX for pulling, validating, parsing and loading
   CSV/TSV and SQL data (ETL)
-- See README.md for full feature description
+* See README.md for full feature description

data/lib/purview/databases/base.rb

@@ -62,7 +62,6 @@ module Purview
       end
 
       def disable_table(table)
-        ensure_table_metadata_exists_for_table(table)
         table_name = table_name(table)
         with_context_logging("`disable_table` for: #{table_name}") do
           with_new_connection do |connection|
@@ -94,7 +93,6 @@ module Purview
       end
 
       def enable_table(table, timestamp=Time.now.utc)
-        ensure_table_metadata_exists_for_table(table)
         table_name = table_name(table)
         with_context_logging("`enable_table` for: #{table_name}") do
           with_new_connection do |connection|
@@ -108,7 +106,6 @@ module Purview
       end
 
       def lock_table(table, timestamp=Time.now.utc)
-        ensure_table_metadata_exists_for_table(table)
         table_name = table_name(table)
         with_context_logging("`lock_table` for: #{table_name}") do
           with_new_connection do |connection|
@@ -142,7 +139,6 @@ module Purview
       end
 
       def unlock_table(table)
-        ensure_table_metadata_exists_for_table(table)
         table_name = table_name(table)
         with_context_logging("`unlock_table` for: #{table_name}") do
           with_new_connection do |connection|
@@ -195,16 +191,6 @@ module Purview
         end
       end
 
-      def connection_opts
-        {
-          :database => database_name,
-          :host => database_host,
-          :password => database_password,
-          :port => database_port,
-          :username => database_username,
-        }
-      end
-
       def connection_type
         raise %{All "#{Base}(s)" must override the "connection_type" method}
       end
@@ -243,7 +229,7 @@ module Purview
       end
 
       def database_host
-        opts[:host]
+        opts[:database_host]
       end
 
       def database_name
@@ -251,15 +237,15 @@ module Purview
       end
 
       def database_password
-        opts[:password]
+        opts[:database_password]
       end
 
       def database_port
-        opts[:port]
+        opts[:database_port]
       end
 
       def database_username
-        opts[:username]
+        opts[:database_username]
       end
 
       def default(column)

data/lib/purview/mixins/connection.rb

@@ -5,6 +5,16 @@ module Purview
         connection_type.new(connection_opts)
       end
 
+      def connection_opts
+        {
+          :database => database_name,
+          :host => database_host,
+          :password => database_password,
+          :port => database_port,
+          :username => database_username,
+        }
+      end
+
       def with_new_connection
         connection_type.with_new_connection(connection_opts) { |connection| yield connection }
       end

data/lib/purview/pullers/base_sql.rb

@@ -17,16 +17,6 @@ module Purview
         table.column_names
       end
 
-      def connection_opts
-        {
-          :database => database_name,
-          :host => database_host,
-          :password => database_password,
-          :port => database_port,
-          :username => database_username,
-        }
-      end
-
       def connection_type
         raise %{All "#{BaseSQL}(s)" must override the "connection_type" method}
       end

data/lib/purview/version.rb

@@ -1,3 +1,3 @@
 module Purview
-  VERSION = '1.0.0'
+  VERSION = '1.0.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: purview
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Jonathan W. Zaleski
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-28 00:00:00.000000000 Z
+date: 2015-05-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler