impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Armstrong (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5307: Part 4: copy out uncompressed text and seq
Date Tue, 31 Oct 2017 21:48:42 GMT
Tim Armstrong has posted comments on this change. ( )

Change subject: IMPALA-5307: Part 4: copy out uncompressed text and seq

Patch Set 8:

File be/src/exec/
PS8, Line 238: 
> nit: a lot of unnecessary blank lines. maybe condense a bit so more code fi
File be/src/exec/
PS8, Line 329:       add_row = WriteCompleteTuple(row_batch->tuple_data_pool(),,
> I missed updating this additional code path, we also need to copy out strin
Done. The below test was previously failing but now succeeds. We should get better coverage
of some of these things once we switch the I/O mgr to the buffer pool and have ASAN poisoning
enabled for recycled buffers.

  ./ -asan -skiptests -noclean -ninja -notests &&
--impalad_args=--disable_mem_pools=true  && impala-py.test -n1 --verbose tests/query_test/
tests/query_test/ --workload_exploration_strategy=functional-query:exhaustive
-k seq
File be/src/exec/
PS8, Line 704:   bool split_delimiter_possible = context_->partition_descriptor()->line_delim()
== '\n'
> I think there's a latent bug here where we're touching byte_buffer_ *after*
I filed IMPALA-6137. I think there are pre-existing bugs here so won't tackle them in this
patch (this patch may have to wait for those though since it increase the odds of hitting

To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I304fd002b61bfedf41c8b1405cd7eb7b492bb941
Gerrit-Change-Number: 8172
Gerrit-PatchSet: 8
Gerrit-Owner: Tim Armstrong <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Michael Ho <>
Gerrit-Reviewer: Tim Armstrong <>
Gerrit-Comment-Date: Tue, 31 Oct 2017 21:48:42 +0000
Gerrit-HasComments: Yes

  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message