Skip to content

Native worker image#2

Open
xpengahana wants to merge 1707 commits into
xpengahana:native-worker-imagefrom
prestodb:native-worker-image
Open

Native worker image#2
xpengahana wants to merge 1707 commits into
xpengahana:native-worker-imagefrom
prestodb:native-worker-image

Conversation

@xpengahana
Copy link
Copy Markdown
Owner

Description

Motivation and Context

Impact

Test Plan

Contributor checklist

  • Please make sure your submission complies with our development, formatting, commit message, and attribution guidelines.
  • PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced.
  • Documented new properties (with its default value), SQL syntax, functions, or other functionality.
  • If release notes are required, they follow the release notes guidelines.
  • Adequate tests were added if applicable.
  • CI passed.

Release Notes

Please follow release notes guidelines and fill in the release notes below.

== RELEASE NOTES ==

General Changes
* ...
* ...

Hive Changes
* ...
* ...

If release note is NOT required, use:

== NO RELEASE NOTE ==

@presto-release-bot presto-release-bot force-pushed the native-worker-image branch 4 times, most recently from daca1b2 to 0effee9 Compare January 11, 2024 02:00
@presto-release-bot presto-release-bot force-pushed the native-worker-image branch 3 times, most recently from 892525f to 571c2c8 Compare January 19, 2024 02:00
@ethanyzhang ethanyzhang force-pushed the native-worker-image branch 3 times, most recently from 15b437d to 076f229 Compare January 20, 2024 05:51
@ethanyzhang ethanyzhang force-pushed the native-worker-image branch 6 times, most recently from 0c1543f to 696144c Compare February 2, 2024 20:39
@presto-release-bot presto-release-bot force-pushed the native-worker-image branch 3 times, most recently from 4b4af20 to 473b334 Compare February 13, 2024 02:01
hantangwangd and others added 30 commits October 16, 2024 08:59
Remove the map to Block conversion of MetricHeader to resolve
the issue with aggregate queries with labels column in Prometheus

Co-authored-by: Yuya Ebihara <ebyhry@gmail.com>
If there is column renaming happening in-between, this will not work.  We need
to fix this in optimizer.
An upgrade to folly introduces the glog headers into the build of
thrift artifacts. This commit adds the glog include directory when building
the targets.
Previously, the code generation step was removed in 7c814ae
However, this makes it more difficult to add new readers in the
future, such as for UUIDs. This change adds the code generation step
as an optional profile which can be enabled when building the
parquet module through -PgenerateParquetReaders
The iceberg spec lists uuid as a valid schema type. Presto supports
UUID types but there was no support for reading or writing them
in the connector.

This commit makes the necessary changes in the connector to create
tables with UUID columns and support for UUIDs in the parquet reader.
This includes an implementation for UUIDs in the batchreader.
… fixes

Summary:
- Add a metric `presto_cpp.memory_pushback_expected_reduction_bytes`
to track expected reduction in memory after a pushback attempt as
this can be different from actual reduction due to other threads
allocating simultaneously
- Fix registeration of a pushback metric
- Make periodicCb() API non-const to allow it to update internal
state if required
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.