Skip to content

Conversation

@aaneja
Copy link
Contributor

@aaneja aaneja commented Nov 3, 2025

Description

Make the spark2 profile inactive by default

Motivation and Context

#26465

Impact

CI will run maven enforcer checks which were accidentally disabled

Test Plan

Ran ./mvnw help:active-profiles and manually confirmed that spark2 is inactive

Contributor checklist

  • Please make sure your submission complies with our contributing guide, in particular code style and commit standards.
  • 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.
  • If adding new dependencies, verified they have an OpenSSF Scorecard score of 5.0 or higher (or obtained explicit TSC approval for lower scores).

Release notes

== NO RELEASE NOTE ==

@aaneja aaneja requested a review from a team as a code owner November 3, 2025 11:19
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Nov 3, 2025
@prestodb-ci prestodb-ci requested review from a team, auden-woolfson and pratyakshsharma and removed request for a team November 3, 2025 11:19
@shrinidhijoshi
Copy link
Collaborator

@aaneja Disabling this effectively makes the presto project unbuildable as there would be unsatisfied dependencies for the presto-spark* modules.

If the main goal is to re-enable the maven enforcer checks that were inadvertantly disabled in this PR #25552, let's work on just re-enabling those checks on the spark2 profile. The original author of the PR Kewen will be working on this.

@aaneja
Copy link
Contributor Author

aaneja commented Nov 5, 2025

The original author of the PR Kewen will be working on this.

Awesome ! Tagging @wangkewen for visibility

Please also fix modules currently failing enforcer, e.g this one - #26019 (comment)

@aaneja
Copy link
Contributor Author

aaneja commented Nov 5, 2025

@shrinidhijoshi Did you try running the Presto build with the spark2 or spark3 profile explicitly set ? (-Pspark2 e.g) The build should work then IMO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

from:IBM PR from IBM

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants