-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature] Bypass sdk read and write decoupling #45918
base: main
Are you sure you want to change the base?
[Feature] Bypass sdk read and write decoupling #45918
Conversation
e0c2c85
to
61d5aad
Compare
d74163f
to
18ed0a7
Compare
be/src/runtime/descriptors.cpp
Outdated
(*tbl)->_slot_with_column_name_map[tdesc.id] = slot_d; | ||
} | ||
|
||
if (!slot_d->col_name().empty()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry
@@ -873,7 +872,7 @@ Status FragmentMgr::exec_external_plan_fragment(const TScanOpenParams& params, c | |||
LOG(WARNING) << "tuple descriptor is null. id: " << slot_ref.tuple_id; | |||
return Status::InvalidArgument("tuple descriptor is null"); | |||
} | |||
auto* slot_desc = desc_tbl->get_slot_descriptor(slot_ref.slot_id); | |||
auto* slot_desc = desc_tbl->get_slot_descriptor_with_column(slot_ref.slot_id); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why change to this new api?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when spark connector using filter push down, be can't recognize slot column name
namespace starrocks::lake { | ||
|
||
extern lake::TabletManager* _lake_tablet_manager; | ||
// extern lake::UpdateManager* _lake_update_manager; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this?
be/src/storage/CMakeLists.txt
Outdated
|
||
if (NOT BUILD_FORMAT_LIB) | ||
list(APPEND STORAGE_FILES lake/starlet_location_provider.cpp) | ||
message(STATUS "include starlet_location_provider.cpp when build Storage.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this?
@@ -103,7 +104,13 @@ const std::shared_ptr<const TabletSchema> Tablet::tablet_schema() const { | |||
} | |||
|
|||
StatusOr<std::shared_ptr<const TabletSchema>> Tablet::get_schema() { | |||
return _mgr->get_tablet_schema(_id, &_version_hint); | |||
if (_tablet_schema) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when will _table_schema not be nullptr?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here Tablet Construct Function has been modified
@@ -239,11 +244,17 @@ StatusOr<TabletMetadataPtr> TabletManager::get_tablet_metadata(int64_t tablet_id | |||
} | |||
|
|||
StatusOr<TabletMetadataPtr> TabletManager::get_tablet_metadata(const string& path, bool fill_cache) { | |||
std::shared_ptr<FileSystem> fs; | |||
return get_tablet_metadata(fs, path, fill_cache); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why pass a invalid fs pointer as a parameter?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove it
a4e1b17
to
0452052
Compare
0452052
to
2c84e0d
Compare
f1ebd72
to
45c4d64
Compare
fc747df
to
635f43f
Compare
[FE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
Signed-off-by: jukejian <[email protected]> Signed-off-by: plotor <[email protected]>
635f43f
to
92db223
Compare
Signed-off-by: Binglin Chang <[email protected]>
Quality Gate passedIssues Measures |
[Java-Extensions Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[BE Incremental Coverage Report]❌ fail : 211 / 320 (65.94%) file detail
|
Why I'm doing:
Fixes #47367
What I'm doing:
You can learn more about the background from pr's #38466.
This PR is mainly used to solve the problem that when BE directly reads and writes SR file format on S3, the code cannot be encapsulated into SDK form. This PR is mainly to decouple some code implementation and provide some compilation scripts.
build format lib
will install directory:
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: