presto-client 0.4.3 → 0.4.4

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    MTY3NjEyOGE1MmMwNmY5ODRjNTEwYjM4ZGQyNWJjNjhhNGEzOTcyNg==
+    MjJiOTFmZDNlNzFmYzlkM2ZkMmE1ZGJkMDkzMjVkMjc5MjA3YzcyMg==
   data.tar.gz: !binary |-
-    YTU3OTgzMDA0NTA2YTM3NDkyZTc5YzM1NDNmMjE5MWYwNjE2OTM5OA==
+    Nzk0NGEzODRlMzBlNjVlNWJkOTZjYWJiZjM2ODc0Y2UxMDBiNGI4Zg==
 SHA512:
   metadata.gz: !binary |-
-    NGE0YjNhZGM0ZGVlNGZiNTQ0NzkzNGZiNWJmMjc3NTE4YWMwYzEzOWRkZTc3
-    NGYwZTg3ZmFiMDk5ZDQ1MGVhMjY0OWZlY2UwOTQxOThhNDY4MTBjNDdiOWRm
-    ZmY1NmE5NTJlNzlhNjg4M2YxYzJiYWFlYWNmNzMxM2ZiMmEyMDk=
+    MGZlNzViNTA4OWVjNmQ2MmRkMTBiN2E0MzBiN2RkMTQzZDI5ZWZjZDBjOTU5
+    ZGU5MGVlZTA4ZjE5OGZkZDYyY2I4ZTY4ZjM0OGUwNmE5MmJlYTBiNjQ2MjE3
+    YTY4NWZkZTVmYmYwMGM3ODBjMzgwYjBiMTgyMTJkMGMzZjk5MGU=
   data.tar.gz: !binary |-
-    OTQ2MDE0N2NkNjhkYzBjZDc4YTFlMjFhMGJmYTRhNmI5NTNjMDljYjhhNTcz
-    YzFlMjUyMTM0NDdlN2Y3MGFkNGZkY2Q0YzkxNmVjOTJhYThiMWNhNzI2Y2M1
-    YzRjYTNhMmZmOWM4ZDVhZThkNTgxYmMyYmQ5ODE1NzU0NjlhNzk=
+    NmMzZmM1NjhmNTI5NDEwODA0MWYzYmM0NjlkYmNmOGExMmRkYTY4MWNlZTQy
+    YzA0ODMyNTE5ZDI5MTQ0ZDQ0ZTk4Yjg5OWNlYTQ4ZTBhZjE0OWIzMDFkZDAz
+    MjllZTdmM2ZlYzZiNjgwOWFkNmRlZmU5OTEzZTRlN2IzMWIwNzg=

data/README.md

@@ -17,7 +17,7 @@ client = Presto::Client.new(
   schema: "default",
   user: "frsyuki",
   time_zone: "US/Pacific", # optional
-  language: "en_us", # optional
+  language: "English", # optional
   properties: {"hello" => "world", "mycatalog.hello" => "world"},  # optional
   http_debug: true,
 )

data/lib/presto/client/query.rb

@@ -48,17 +48,23 @@ module Presto::Client
       @api.advance
     end
 
+    def wait_for_columns
+      while @api.current_results.columns == nil && @api.advance
+      end
+    end
+
     def wait_for_data
       while @api.current_results.data == nil && @api.advance
       end
     end
 
+    private :wait_for_columns
     private :wait_for_data
 
     def columns
-      wait_for_data
+      wait_for_columns
 
-      raise_error unless @api.query_succeeded?
+      raise_if_failed
 
       return @api.current_results.columns
     end
@@ -80,7 +86,7 @@ module Presto::Client
     def each_row_chunk(&block)
       wait_for_data
 
-      raise_error unless @api.query_succeeded?
+      raise_if_failed
 
       if self.columns == nil
         raise PrestoError, "Query #{@api.current_results.id} has no columns"
@@ -106,7 +112,7 @@ module Presto::Client
       nil
     end
 
-    def raise_error
+    def raise_if_failed
       if @api.closed?
         raise PrestoClientError, "Query aborted by user"
       elsif @api.exception?
@@ -118,8 +124,6 @@ module Presto::Client
         raise PrestoQueryError.new("Query #{results.id} failed: #{error.message}", results.id, error.error_code, error.failure_info)
       end
     end
-
-    private :raise_error
   end
 
 end

data/lib/presto/client/version.rb

@@ -15,6 +15,6 @@
 #
 module Presto
   module Client
-    VERSION = "0.4.3"
+    VERSION = "0.4.4"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: presto-client
 version: !ruby/object:Gem::Version
-  version: 0.4.3
+  version: 0.4.4
 platform: ruby
 authors:
 - Sadayuki Furuhashi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-20 00:00:00.000000000 Z
+date: 2014-12-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday