Skip to content

Commit

Permalink
[#11720] DocDB: Add ability to evaluate expression using lightweight …
Browse files Browse the repository at this point in the history
…protobufs

Summary:
Currenly we could evaluate expression represented as google protobufs.
This diff adds ability to evaluate expression represented as lightweight protobufs.

Test Plan: Jenkins

Reviewers: bogdan, dmitry

Reviewed By: bogdan, dmitry

Subscribers: ybase

Differential Revision: https://phabricator.dev.yugabyte.com/D16221
  • Loading branch information
spolitov committed Mar 29, 2022
1 parent 5baeb16 commit 801251f
Show file tree
Hide file tree
Showing 41 changed files with 1,343 additions and 1,016 deletions.
3 changes: 1 addition & 2 deletions python/yb/lto.py
Original file line number Diff line number Diff line change
Expand Up @@ -298,8 +298,7 @@ def add_leaf_object_files(self) -> None:
# Dedup .cc.o files already existing on the command line.
continue

if (node.node_type == NodeType.OBJECT and
os.path.basename(os.path.dirname(node.path)) != 'yb_common_base.dir'):
if node.node_type == NodeType.OBJECT:
self.new_args.add_new_arg(node.path)

for arg in self.yb_pgbackend_link_cmd:
Expand Down
5 changes: 3 additions & 2 deletions src/yb/bfpg/bfpg-test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,11 @@ class BFTestValue : public QLValue {
ql_type_id_ = DataType::UNKNOWN_DATA;
}

virtual DataType ql_type_id() const {
DataType ql_type_id() const {
return ql_type_id_;
}
virtual void set_ql_type_id(DataType ql_type_id) {

void set_ql_type_id(DataType ql_type_id) {
ql_type_id_ = ql_type_id;
}

Expand Down
Loading

0 comments on commit 801251f

Please sign in to comment.