Skip to content
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

BACKPORT-TO-STABLE: Backported fixes for Drill 1.21.2 #2860

Merged
merged 49 commits into from
Apr 12, 2024

Conversation

jnturton
Copy link
Contributor

@jnturton jnturton commented Dec 31, 2023

[BACKPORT-TO-STABLE]: fixes for Drill 1.21.2

Description

Fixes and minor enhancements since Drill 1.21.1 backported from the master branch.

@cgivre
Copy link
Contributor

cgivre commented Jan 1, 2024

@jnturton
Did we add #2795 to this?

@jnturton
Copy link
Contributor Author

jnturton commented Jan 2, 2024

@jnturton Did we add #2795 to this?

@cgivre yes, it's here.

Copy link
Member

@luocooong luocooong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

cgivre and others added 21 commits January 3, 2024 10:40
* chore(deps): Update Yauaa to 7.19.2

* Additional fix, please squash on merge
Bumps [h2](https://github.com/h2database/h2database) from 2.1.210 to 2.2.220.
- [Release notes](https://github.com/h2database/h2database/releases)
- [Commits](h2database/h2database@version-2.1.210...version-2.2.220)

---
updated-dependencies:
- dependency-name: com.h2database:h2
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…apache#2822)

* Shut down MiniDFS cluster in TestInboundImpersonation.selectChainedView.

* Ensure that MiniDFS is using simple auth in BaseTestImpersonation.
…pache#2839)

* Add a 2GB swap file when setting up the Actions Runner.
* Remove /proc/sys/vm/drop_caches snake oil.
* Tune the heap and direct memory limits applied to the test suite.
pjfanning and others added 5 commits January 3, 2024 10:40
…apache#2861)

The EasyOutOfMemory test category is a clever workaround to run Drill unit tests in the memory constrained CI
environment provided by Github Actions that has served us well but that is no longer needed now that
we set up swap space in the CI workflow. We can retire it for simpler and faster CI testing.
This increase is not applicable to master where the removal of Drill's shaded
Guava has shrunk the JDBC driver JAR size.
* Upgrade Janino 3.1.8 → 3.1.11.
* Update classb/ and 3rdparty/ lib separation for jakarta packages.
apache#2865)

* Upgrade ZooKeeper 3.5.7 → 3.5.10 and Curator 5.2.0 → 5.5.0
@cgivre
Copy link
Contributor

cgivre commented Jan 3, 2024

@jnturton
I'd like to include DRILL-8470, and possibly DRILL-8415 and DRILL-8465 in this. What do you think? I'm on the fence about DRILL-8465 as that does seem like more of a new feature.

cgivre and others added 4 commits January 5, 2024 08:13
* Upgrade MongoDB driver 4.3.3 → 4.11.1
* Upgrade DeltaLake 2.1.0 → 3.0.0
* Upgrade Metadata-Extractor 2.18.0 → 2.19.0
Allows the readers using the EVF layer to write to
UNION and LIST vectors whether or not those vectors
are projected by the query. EVF creates "dummy" vectors
and writers for non-projected columns. This feature
previously worked for other column types: this PR adds
support for UNION and LIST.
* Upgrade Jackson 2.14.3 → 2.16.1.
* Do some list sorting in java-exec's drill-module.conf.
* Bump JDBC jar size to 54.5Mb.
@jnturton
Copy link
Contributor Author

@jnturton I'd like to include DRILL-8470, and possibly DRILL-8415 and DRILL-8465 in this. What do you think? I'm on the fence about DRILL-8465 as that does seem like more of a new feature.

8415 and 8470 are here. 8465 is still in analysis and design. We still need the new 8475 (NOTICE file incomplete) that I marked as a blocker and which is being worked on by @CalvinKirs and me.

@cgivre
Copy link
Contributor

cgivre commented Jan 10, 2024

@jnturton I'd like to include DRILL-8470, and possibly DRILL-8415 and DRILL-8465 in this. What do you think? I'm on the fence about DRILL-8465 as that does seem like more of a new feature.

8415 and 8470 are here. 8465 is still in analysis and design. We still need the new 8475 (NOTICE file incomplete) that I marked as a blocker and which is being worked on by @CalvinKirs and me.

I think we should drop 8465 from 1.21.2. It sounds like once the notices are done we should be ready to cut a RC!

@jnturton jnturton merged commit 0e6b21d into apache:1.21 Apr 12, 2024
0 of 5 checks passed
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.