Skip to content

Commit 3abeb5e

Browse files
fixes
1 parent 0c6af53 commit 3abeb5e

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

ydb/core/formats/arrow/program/filter.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,8 @@ class TFilterVisitor: public arrow::ArrayVisitor {
6060
arrow::Status VisitImpl(const TArray& array) {
6161
AFL_VERIFY(Started);
6262
for (ui32 i = 0; i < array.length(); ++i) {
63-
const bool columnValue = !array.IsNull(i) && (bool)array.Value(i);
6463
const ui32 currentIdx = CursorIdx++;
65-
FiltersMerged[currentIdx] = FiltersMerged[currentIdx] && columnValue;
64+
FiltersMerged[currentIdx] = FiltersMerged[currentIdx] && !array.IsNull(i) && (bool)array.Value(i);
6665
}
6766
AFL_VERIFY(CursorIdx <= FiltersMerged.size());
6867
return arrow::Status::OK();

ydb/core/formats/arrow/program/graph.cpp

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,13 +212,19 @@ TConclusion<std::vector<std::shared_ptr<IResourceProcessor>>> TGraph::BuildChain
212212
if (readyNodeIds.size() != Nodes.size()) {
213213
std::set<ui32> notCoveredIds;
214214
TStringBuilder sb;
215+
ui32 count = 0;
215216
for (auto&& [id, n] : Nodes) {
216217
if (!readyNodeIds.contains(id)) {
218+
if (n->GetProcessor()->GetProcessorType() != EProcessorType::Const) {
219+
++count;
220+
}
217221
sb << n->DebugJson().GetStringRobust() << "/" << n->GetProcessor()->DebugJson().GetStringRobust() << Endl;
218222
}
219223
}
220-
return TConclusionStatus::Fail(
221-
"not found final nodes: " + ::ToString(readyNodeIds.size()) + " covered from " + ::ToString(Nodes.size()) + ": details = " + sb);
224+
if (count) {
225+
return TConclusionStatus::Fail(
226+
"not found final nodes: " + ::ToString(readyNodeIds.size()) + " covered from " + ::ToString(Nodes.size()) + ": details = " + sb);
227+
}
222228
}
223229
std::vector<std::shared_ptr<IResourceProcessor>> result;
224230
for (auto&& c : nodeChains) {

0 commit comments

Comments
 (0)