trino-client 2.2.4 → 2.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86c798d4b43a4371f78cd8dac5e19c62faef5a0d8e5d486a9afe05c94b181fba
-  data.tar.gz: e19a05e46624e829c3c9e2fe0588875d86c9d3bb6cce53f178079bc949b4c20c
+  metadata.gz: 69f501fb237b50a19cab4a4ca36e5702ce0b6852aa4f47c28cc58ee2bf9b0b56
+  data.tar.gz: 0505db32b400bd17516b49b7c151c5717a95cd6316f3a0a91258409e483822c3
 SHA512:
-  metadata.gz: 90c0f9d87a0c0e038f90b91ad13c522b828930ed6d13185a743613d4a7e955c77c180e368ae2a167cec12ab477fad5534768bf15e14be7ca56c20f8f7cb8b936
-  data.tar.gz: 54aa3d204e9db250e5108f2e3e21f140cc546def4a01911ba71a8dd9a54eb5ccdeadc036a6073a33f8f90fc6e73225d2d65065609729461e8daebb502513ca24
+  metadata.gz: f31ffb1267de01b418f3a39c244dcf34dc8f3dd2d5d3bff3e15c6dc6b8aef1c701ec70ac46c5b85f08a1de89094e0be0198439083e9a57b337bb72c6c3f549e7
+  data.tar.gz: 77aeab61775faadb4cd79f567089cb3cfd25c86e3a6d5469ae82da589ad2b88ca59f5e982c49d9aba3832960dc50c1236246913098bc1422587e80cdebb1e475

data/ChangeLog.md

@@ -1,5 +1,31 @@
 trino-client-ruby
 ====
+## 2.2.6
+- fix: improve POST retry tests and document idempotency assumption ([#159](https://github.com/treasure-data/trino-client-ruby/issues/159)) [[e64c2a9](https://github.com/treasure-data/trino-client-ruby/commit/e64c2a9)]
+- Add retry to POST /v1/statement ([#157](https://github.com/treasure-data/trino-client-ruby/issues/157)) [[64a5c66](https://github.com/treasure-data/trino-client-ruby/commit/64a5c66)]
+- Update psych requirement from ~> 3 to ~> 5 ([#140](https://github.com/treasure-data/trino-client-ruby/issues/140)) [[eeb3333](https://github.com/treasure-data/trino-client-ruby/commit/eeb3333)]
+- Update dependabot.yaml to change the update interval from weekly to monthly and set commit message prefix to "chore". [[2fde3f2](https://github.com/treasure-data/trino-client-ruby/commit/2fde3f2)]
+- Minor fix [[db03c07](https://github.com/treasure-data/trino-client-ruby/commit/db03c07)]
+- Update addressable requirement from ~> 2.8.1 to ~> 2.9.0 [[1a56064](https://github.com/treasure-data/trino-client-ruby/commit/1a56064)]
+- Update README.md for release instructions [skip ci] [[dda6a06](https://github.com/treasure-data/trino-client-ruby/commit/dda6a06)]
+- v2.2.5 [[12ce8e8](https://github.com/treasure-data/trino-client-ruby/commit/12ce8e8)]
+
+## 2.2.5
+- Merge pull request [#155](https://github.com/treasure-data/trino-client-ruby/issues/155) from murakami-ta/add-http-debug-logger-option [[143a801](https://github.com/treasure-data/trino-client-ruby/commit/143a801)]
+- Fix http_debug_logger test to work with both Faraday 1 and 2 [[0976be7](https://github.com/treasure-data/trino-client-ruby/commit/0976be7)]
+- Add http_debug_logger option [[b54b20a](https://github.com/treasure-data/trino-client-ruby/commit/b54b20a)]
+- Bump rubygems/release-gem from 1.1.4 to 1.2.0 ([#153](https://github.com/treasure-data/trino-client-ruby/issues/153)) [[52e0e30](https://github.com/treasure-data/trino-client-ruby/commit/52e0e30)]
+- Bump release-drafter/release-drafter from 6 to 7 ([#152](https://github.com/treasure-data/trino-client-ruby/issues/152)) [[851d586](https://github.com/treasure-data/trino-client-ruby/commit/851d586)]
+- Bump rubygems/release-gem from 1.1.2 to 1.1.4 ([#151](https://github.com/treasure-data/trino-client-ruby/issues/151)) [[bca14ae](https://github.com/treasure-data/trino-client-ruby/commit/bca14ae)]
+- Update standard requirement from ~> 1.51.1 to ~> 1.54.0 ([#150](https://github.com/treasure-data/trino-client-ruby/issues/150)) [[9dc220e](https://github.com/treasure-data/trino-client-ruby/commit/9dc220e)]
+- Bump actions/checkout from 5 to 6 ([#147](https://github.com/treasure-data/trino-client-ruby/issues/147)) [[76f9954](https://github.com/treasure-data/trino-client-ruby/commit/76f9954)]
+- Bump github/codeql-action from 3 to 4 ([#145](https://github.com/treasure-data/trino-client-ruby/issues/145)) [[3065e2a](https://github.com/treasure-data/trino-client-ruby/commit/3065e2a)]
+- Update standard requirement from ~> 1.50.0 to ~> 1.51.1 ([#143](https://github.com/treasure-data/trino-client-ruby/issues/143)) [[be846e0](https://github.com/treasure-data/trino-client-ruby/commit/be846e0)]
+- Bump rubygems/release-gem ([#146](https://github.com/treasure-data/trino-client-ruby/issues/146)) [[9821efc](https://github.com/treasure-data/trino-client-ruby/commit/9821efc)]
+- Add DeepWiki badge to README [skip ci] [[82afa57](https://github.com/treasure-data/trino-client-ruby/commit/82afa57)]
+- Update release.rb to increment the version in @lib/trino/client/version.rb [[c2bff61](https://github.com/treasure-data/trino-client-ruby/commit/c2bff61)]
+- Bump actions/checkout from 4 to 5, etc ([#142](https://github.com/treasure-data/trino-client-ruby/issues/142)) [[17eb1d2](https://github.com/treasure-data/trino-client-ruby/commit/17eb1d2)]
+
 ## 2.2.4
 - Add steps to publish trino-client-ruby [[228f709](https://github.com/treasure-data/trino-client-ruby/commit/228f709)]
 

data/README.md

@@ -1,6 +1,7 @@
 # Trino client library for Ruby
 
 [![Ruby](https://github.com/treasure-data/trino-client-ruby/actions/workflows/ruby.yml/badge.svg)](https://github.com/treasure-data/trino-client-ruby/actions/workflows/ruby.yml) [![Gem](https://img.shields.io/gem/v/trino-client)](https://rubygems.org/gems/trino-client) [![Gem](https://img.shields.io/gem/dt/trino-client)](https://rubygems.org/gems/trino-client) [![GitHub](https://img.shields.io/github/license/treasure-data/trino-client-ruby)]()
+[![Ask DeepWiki](https://deepwiki.com/badge.svg)](https://deepwiki.com/treasure-data/trino-client-ruby)
 
 Trino is a distributed SQL query engine for big data:
 https://trino.io/
@@ -112,6 +113,7 @@ $ bundle exec rake modelgen:latest
 * **http_headers** sets custom HTTP headers. It must be a Hash of string to string.
 * **http_proxy** sets host:port of a HTTP proxy server.
 * **http_debug** enables debug message to STDOUT for each HTTP requests.
+* **http_debug_logger** sets a custom `Logger` instance for HTTP debug logs. Requires **http_debug** to be `true`.
 * **http_open_timeout** sets timeout in seconds to open new HTTP connection.
 * **http_timeout** sets timeout in seconds to read data from a server.
 * **gzip** enables gzip compression.
@@ -124,20 +126,17 @@ See [RDoc](http://www.rubydoc.info/gems/presto-client/) for the full documentati
 
 ### Releasing a new version
 
-1. First update `lib/trino/client/version.rb` to the next version.
-2. Run the following command which will update `ChangeLog.md` file automatically.
+1. Run the following command which will prompt for the new version, update `lib/trino/client/version.rb`, and generate/let you edit `ChangeLog.md`.
 ```
 $ ruby release.rb
 ```
 
-3. Create tag
+2. Commit and tag
 ```
 $ git commit -am "vX.Y.Z"
 $ git tag "vX.Y.Z"
-% git push --tags
+$ git push origin master
+$ git push --tags
 ```
 
-4. Push package by the following command which will build and push `trino-client-X.Y.Z.gem` and `trino-client-ruby-X.Y.Z.gem` automatically.
-```
-$ ruby publish.rb
-```
+Pushing a `v*` tag triggers the GitHub Actions publisher workflow, which automatically builds and publishes the gem to RubyGems.org.

data/lib/trino/client/faraday_client.rb

@@ -91,7 +91,7 @@ module Trino::Client
       if options[:gzip]
         faraday.request :gzip
       end
-      faraday.response :logger if options[:http_debug]
+      faraday.response :logger, options[:http_debug_logger] if options[:http_debug]
       faraday.adapter Faraday.default_adapter
     end
 

data/lib/trino/client/statement_client.rb

@@ -26,6 +26,8 @@ module Trino::Client
         :max_nesting => false
     }
 
+    RETRYABLE_STATUSES = [502, 503, 504]
+
     def initialize(faraday, query, options, next_uri=nil)
       @faraday = faraday
 
@@ -66,16 +68,31 @@ module Trino::Client
 
     def post_query_request!
       uri = "/v1/statement"
-      response = @faraday.post do |req|
-        req.url uri
+      response = with_retry_loop do
+        begin
+          r = @faraday.post do |req|
+            req.url uri
 
-        req.body = @query
-        init_request(req)
-      end
+            req.body = @query
+            init_request(req)
+          end
+        rescue Faraday::TimeoutError, Faraday::ConnectionFailed
+          # POST /v1/statement is not idempotent; retrying transport-level errors
+          # (timeouts/connection failures) can submit duplicate queries. This is
+          # intentional under the assumption the failure happened before Trino
+          # received the request (e.g. rejected by a load balancer upstream).
+          throw :retry_with_backoff
+        rescue => e
+          exception! e
+        end
 
-      # TODO error handling
-      if response.status != 200
-        exception! TrinoHttpError.new(response.status, "Failed to start query: #{response.body} (#{response.status})")
+        if r.status == 200
+          r
+        elsif RETRYABLE_STATUSES.include?(r.status)
+          throw :retry_with_backoff
+        else
+          exception! TrinoHttpError.new(r.status, "Failed to start query: #{r.body} (#{r.status})")
+        end
       end
 
       @results_headers = response.headers
@@ -189,31 +206,12 @@ module Trino::Client
 
     private :parse_body
 
-    def faraday_get_with_retry(uri, &block)
+    def with_retry_loop
       start = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       attempts = 0
 
       loop do
-        begin
-          response = @faraday.get(uri)
-        rescue Faraday::TimeoutError, Faraday::ConnectionFailed
-          # temporally error to retry
-          response = nil
-        rescue => e
-          exception! e
-        end
-
-        if response
-          if response.status == 200 && !response.body.to_s.empty?
-            return response
-          end
-
-          # retry if 502, 503, 504 according to the trino protocol
-          unless [502, 503, 504].include?(response.status)
-            # deterministic error
-            exception! TrinoHttpError.new(response.status, "Trino API error at #{uri} returned #{response.status}: #{response.body}")
-          end
-        end
+        catch(:retry_with_backoff) { return yield }
 
         raise_if_timeout!
 
@@ -226,6 +224,28 @@ module Trino::Client
       exception! TrinoHttpError.new(408, "Trino API error due to timeout")
     end
 
+    private :with_retry_loop
+
+    def faraday_get_with_retry(uri)
+      with_retry_loop do
+        begin
+          response = @faraday.get(uri)
+        rescue Faraday::TimeoutError, Faraday::ConnectionFailed
+          throw :retry_with_backoff
+        rescue => e
+          exception! e
+        end
+
+        if response.status == 200 && !response.body.to_s.empty?
+          response
+        elsif RETRYABLE_STATUSES.include?(response.status)
+          throw :retry_with_backoff
+        else
+          exception! TrinoHttpError.new(response.status, "Trino API error at #{uri} returned #{response.status}: #{response.body}")
+        end
+      end
+    end
+
     def raise_if_timeout!
       if @started_at
         return if finished?

data/lib/trino/client/version.rb

@@ -15,6 +15,6 @@
 #
 module Trino
   module Client
-    VERSION = "2.2.4"
+    VERSION = "2.2.6"
   end
 end

data/trino-client.gemspec

@@ -29,8 +29,8 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency "rake", [">= 0.9.2", "< 14.0"]
   gem.add_development_dependency "rspec", "~> 3.13.0"
   gem.add_development_dependency "webmock", ["~> 3.0"]
-  gem.add_development_dependency "addressable", "~> 2.8.1" # 2.5.0 doesn't support Ruby 1.9.3
+  gem.add_development_dependency "addressable", "~> 2.9.0"
   gem.add_development_dependency "simplecov", "~> 0.22.0"
-  gem.add_development_dependency "standard", "~> 1.50.0"
-  gem.add_development_dependency "psych", "~> 3"
+  gem.add_development_dependency "standard", "~> 1.54.0"
+  gem.add_development_dependency "psych", "~> 5"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: trino-client
 version: !ruby/object:Gem::Version
-  version: 2.2.4
+  version: 2.2.6
 platform: ruby
 authors:
 - Sadayuki Furuhashi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-09-11 00:00:00.000000000 Z
+date: 2026-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -126,14 +126,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.8.1
+        version: 2.9.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.8.1
+        version: 2.9.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -154,28 +154,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.50.0
+        version: 1.54.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.50.0
+        version: 1.54.0
 - !ruby/object:Gem::Dependency
   name: psych
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '5'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '5'
 description: Trino client library
 email:
 - sf@treasure-data.com