Skip to content

fix: Add missing name field in presto-sql-helper plugins#26307

Merged
unidevel merged 1 commit intoprestodb:release-0.295from
unidevel:fix-name
Oct 15, 2025
Merged

fix: Add missing name field in presto-sql-helper plugins#26307
unidevel merged 1 commit intoprestodb:release-0.295from
unidevel:fix-name

Conversation

@unidevel
Copy link
Contributor

Description

Error when publishing maven plugins, logs:

Deployment 1a2273ac-3e3d-4502-aeca-72658dda57f6 failed
pkg:maven/com.facebook.presto/presto-native-sql-invoked-functions-plugin@0.295:
 - Project name is missing

pkg:maven/com.facebook.presto/presto-sql-invoked-functions-plugin@0.295:
 - Project name is missing

Motivation and Context

Impact

0.295-release

Test Plan

Release

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.

Release Notes

== NO RELEASE NOTE ==

@unidevel unidevel requested a review from a team as a code owner October 14, 2025 20:13
@prestodb-ci prestodb-ci added the from:IBM PR from IBM label Oct 14, 2025
@prestodb-ci prestodb-ci requested review from a team, Dilli-Babu-Godari and namya28 and removed request for a team October 14, 2025 20:13
@unidevel unidevel requested review from czentgr and tdcmeehan October 14, 2025 20:14
@tdcmeehan
Copy link
Contributor

Thanks @unidevel

@unidevel
Copy link
Contributor Author

Thanks @tdcmeehan , I’ll investigate how to validate this for pull requests later.

@unidevel unidevel changed the title Fix missing name field in presto-sql-helper plugins fix: Add missing name field in presto-sql-helper plugins Oct 14, 2025
@unidevel
Copy link
Contributor Author

I will also create another PR to master branch after the maven artifacts published without error.

@unidevel unidevel merged commit a899b6c into prestodb:release-0.295 Oct 15, 2025
76 of 77 checks passed
unidevel added a commit to unidevel/presto that referenced this pull request Oct 15, 2025
)

## Description

Error when publishing maven plugins, logs:

```log
Deployment 1a2273ac-3e3d-4502-aeca-72658dda57f6 failed
pkg:maven/com.facebook.presto/presto-native-sql-invoked-functions-plugin@0.295:
 - Project name is missing

pkg:maven/com.facebook.presto/presto-sql-invoked-functions-plugin@0.295:
 - Project name is missing
 ```
## Motivation and Context

## Impact
0.295-release

## Test Plan
Release

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md), in particular [code style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style) and [commit standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#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](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.

## Release Notes
```
== NO RELEASE NOTE ==
```
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.

4 participants