vertica 0.9.3 → 0.9.4

data/README.md

@@ -5,7 +5,7 @@ about Vertica at http://www.vertica.com.
 
 This library currently supports connecting, executing SQL queries, and transferring data
 for a "COPY table FROM STDIN" statement. The gem is tested against Vertica version 4.1, 
-5.0, and 5.1, and Ruby version 1.8 and 1.9.
+5.0, 5.1, and 6.1, and Ruby version 1.8 and 1.9.
 
 # Install
 
@@ -56,7 +56,12 @@ Get all the result rows without buffering by providing a block:
     end
     
     connection.close
-    
+
+Note: you can only use the connection for one query at the time. If you try to run another 
+query when the connection is still busy delivering the results of a previous query, a
+`Vertica::Error::SynchronizeError` will be raised. Use buffered resultsets to prevent this
+problem.
+
 ### Buffered result
 
 Store the result of the query method as a variable to get a buffered resultset:
@@ -108,3 +113,4 @@ prefixed with <tt>test_ruby_vertica_</tt>.
  * [Matt Bauer](http://github.com/mattbauer) all the hard work
  * [Jeff Smick](http://github.com/sprsquish) current maintainer
  * [Willem van Bergen](http://github.com/wvanbergen) contributor
+ * [Camilo Lopez](http://github.com/camilo) contributor

data/VERSION

@@ -1 +1 @@
-0.9.3
+0.9.4

data/lib/vertica/connection.rb

@@ -41,7 +41,7 @@ class Vertica::Connection
           raw_socket.sync = true
           raw_socket.connect
         else
-          raise Vertica::Error::ConnectionError.new("SSL requested but server doesn't support it.")
+          raise Vertica::Error::SSLNotSupported.new("SSL requested but server doesn't support it.")
         end
       end
       
@@ -99,7 +99,7 @@ class Vertica::Connection
   end
 
   def interrupt
-    raise Vertica::Error::ConnectionError, "Session cannopt be interrupted because the session ID is not known!" if session_id.nil?
+    raise Vertica::Error::InterruptImpossible, "Session cannopt be interrupted because the session ID is not known!" if session_id.nil?
     conn = self.class.new(options.merge(:interruptable => false, :role => nil, :search_path => nil))
     response = conn.query("SELECT CLOSE_SESSION(#{Vertica.quote(session_id)})").the_value
     conn.close

data/lib/vertica/error.rb

@@ -2,6 +2,8 @@
 class Vertica::Error < StandardError
   
   class ConnectionError < Vertica::Error; end
+  class SSLNotSupported < ConnectionError; end
+  class InterruptImpossible < Vertica::Error; end
   class MessageError < Vertica::Error; end
   class SynchronizeError < Vertica::Error; end
   class EmptyQueryError < Vertica::Error; end
@@ -32,6 +34,7 @@ class Vertica::Error < StandardError
     '42601' => (SyntaxError           = Class.new(Vertica::Error::QueryError)),
     '42V01' => (MissingRelation       = Class.new(Vertica::Error::QueryError)),
     '42703' => (MissingColumn         = Class.new(Vertica::Error::QueryError)),
-    '22V04' => (CopyRejected          = Class.new(Vertica::Error::QueryError))
+    '22V04' => (CopyRejected          = Class.new(Vertica::Error::QueryError)),
+    '42501' => (PermissionDenied      = Class.new(Vertica::Error::QueryError))
   }
 end

data/test/functional/connection_test.rb

@@ -6,65 +6,65 @@ class ConnectionTest < Test::Unit::TestCase
     @connection.close if @connection
   end
 
-  def test_new_connection
-    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
+  def assert_valid_open_connection(connection)
+    assert connection.opened?
+    assert !connection.closed?
 
-    assert !@connection.parameters.empty?
-    assert @connection.backend_pid
-    assert @connection.backend_key
-    assert @connection.transaction_status
-    assert @connection.opened?
-    assert !@connection.closed?
+    # connection variables
+    assert connection.backend_pid
+    assert connection.backend_key
+    assert connection.transaction_status
     
     # parameters
-    assert @connection.parameters.kind_of?(Hash)
-    assert @connection.parameters.include?('server_version')
+    assert connection.parameters.kind_of?(Hash)
+    assert connection.parameters.include?('server_version')
   end
-  
-  def test_close_connection
-    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
-    @connection.close
 
-    assert !@connection.opened?
-    assert @connection.closed?
-    assert_equal({}, @connection.parameters)
-    assert_nil @connection.backend_pid
-    assert_nil @connection.backend_key
-    assert_nil @connection.transaction_status
+  def assert_valid_closed_connection(connection)
+    assert !connection.opened?
+    assert connection.closed?
+    assert_equal({}, connection.parameters)
+    assert_nil connection.backend_pid
+    assert_nil connection.backend_key
+    assert_nil connection.transaction_status
+  end
+
+  def test_opening_and_closing_connection
+    connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
+    assert_valid_open_connection(connection)
+
+    connection.close
+    assert_valid_closed_connection(connection)    
   end
   
   def test_connection_with_ssl
-    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH.merge(:ssl => true))
+    connection = Vertica::Connection.new(TEST_CONNECTION_HASH.merge(:ssl => true))
+    assert_valid_open_connection(connection)
+    assert connection.ssl?
 
-    assert @connection.ssl?
-    assert !@connection.parameters.empty?
-    assert @connection.backend_pid
-    assert @connection.backend_key
-    assert @connection.transaction_status
-
-    @connection.close
+    connection.close
+    assert_valid_closed_connection(connection)
+    assert !connection.ssl?
     
-    assert_equal({}, @connection.parameters)
-    assert_nil @connection.backend_pid
-    assert_nil @connection.backend_key
-    assert_nil @connection.transaction_status
+  rescue Vertica::Error::SSLNotSupported => e
+    puts "\nThe test server doesn't support SSL, so SSL connections could not be tested."
   end
 
   def test_reset_connection
-    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
-    original_backend_pid = @connection.backend_pid
-    original_backend_key = @connection.backend_key
+    connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
+    original_backend_pid, original_backend_key = connection.backend_pid, connection.backend_key
 
-    @connection.reset_connection
+    connection.reset_connection
 
-    assert_not_equal original_backend_pid, @connection.backend_pid
-    assert_not_equal original_backend_key, @connection.backend_key
-    assert_equal :no_transaction, @connection.transaction_status
+    assert_valid_open_connection(connection)
+    assert_not_equal original_backend_pid, connection.backend_pid
+    assert_not_equal original_backend_key, connection.backend_key
+    assert_equal :no_transaction, connection.transaction_status
   end
   
-  def test_interrupt_connection
-    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH.merge(:interruptable => true))
-    assert @connection.interruptable?, "The connection should be interruptable!"
+  def test_interruptable_connection
+    connection = Vertica::Connection.new(TEST_CONNECTION_HASH.merge(:interruptable => true))
+    assert connection.interruptable?, "The connection should be interruptable!"
   end
 
   def test_new_with_error_response

data/test/functional/query_test.rb

@@ -193,7 +193,7 @@ class QueryTest < Test::Unit::TestCase
     assert_equal values.sort, [1,1,1,2,2,2,3,3,3]
   end
   
-  def test_raise_on_synchronisity_problems
+  def test_raise_when_connection_is_in_use
     assert_raise(Vertica::Error::SynchronizeError) do
       @connection.query("SELECT 1 UNION SELECT 2") do |record|
         @connection.query("SELECT 3")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vertica
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.4
   prerelease: 
 platform: ruby
 authors:
@@ -11,11 +11,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-10 00:00:00.000000000 Z
+date: 2013-01-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70275673064020 !ruby/object:Gem::Requirement
+  requirement: &70135370266420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -23,10 +23,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70275673064020
+  version_requirements: *70135370266420
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70275673063540 !ruby/object:Gem::Requirement
+  requirement: &70135370265940 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -34,7 +34,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70275673063540
+  version_requirements: *70135370265940
 description: Query Vertica with ruby
 email: sprsquish@gmail.com
 executables: []