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)))