You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by tu...@apache.org on 2024/09/06 05:26:39 UTC

(arrow-rs) branch master updated: object_store/delimited: Fix `TrailingEscape` condition (#6265)

This is an automated email from the ASF dual-hosted git repository.

tustvold pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git


The following commit(s) were added to refs/heads/master by this push:
     new 6da4793347 object_store/delimited: Fix `TrailingEscape` condition (#6265)
6da4793347 is described below

commit 6da4793347d4217cce7ecc88799faa609ac5adcb
Author: Tobias Bieniek <to...@bieniek.cloud>
AuthorDate: Fri Sep 6 07:26:24 2024 +0200

    object_store/delimited: Fix `TrailingEscape` condition (#6265)
    
    This seems like a copy-paste mistake since checking `is_quote` twice is probably wrong...
---
 object_store/src/delimited.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/object_store/src/delimited.rs b/object_store/src/delimited.rs
index 4f25c9d6d3..96f88bf41f 100644
--- a/object_store/src/delimited.rs
+++ b/object_store/src/delimited.rs
@@ -126,7 +126,7 @@ impl LineDelimiter {
     fn finish(&mut self) -> Result<bool> {
         if !self.remainder.is_empty() {
             ensure!(!self.is_quote, UnterminatedStringSnafu);
-            ensure!(!self.is_quote, TrailingEscapeSnafu);
+            ensure!(!self.is_escape, TrailingEscapeSnafu);
 
             self.complete
                 .push_back(Bytes::from(std::mem::take(&mut self.remainder)))