Skip to content

MINOR: Re-introduce Transformation import to fix TransformationChain Javadoc - #13808

Merged
C0urante merged 1 commit into
apache:trunkfrom
yashmayya:fix-transformation-chain-javadoc
Jun 5, 2023
Merged

MINOR: Re-introduce Transformation import to fix TransformationChain Javadoc#13808
C0urante merged 1 commit into
apache:trunkfrom
yashmayya:fix-transformation-chain-javadoc

Conversation

@yashmayya

Copy link
Copy Markdown
Contributor

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@yashmayya
yashmayya requested a review from C0urante June 5, 2023 11:30

@C0urante C0urante left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks Yash! LGTM

@C0urante

C0urante commented Jun 5, 2023

Copy link
Copy Markdown
Contributor

Not sure what's going on with Jenkins but I've verified locally that this builds correctly and fixes the relevant Javadocs linking issue. Merging...

@C0urante
C0urante merged commit 8e60368 into apache:trunk Jun 5, 2023
@yashmayya

Copy link
Copy Markdown
Contributor Author

Thanks Chris! Yeah, the Jenkins thing is odd - no build has been triggered for this merged commit on trunk either. Do we need to raise an Apache INFRA Jira ticket?

@yashmayya
yashmayya deleted the fix-transformation-chain-javadoc branch June 5, 2023 13:52
@C0urante

C0urante commented Jun 5, 2023

Copy link
Copy Markdown
Contributor

Wild guess with little backing knowledge: I wonder if the queue is just full. It looks like there's two open PRs right now that don't have finished CI runs; the one that was opened earlier is currently running, and the most-recently-opened one doesn't have anything (yet).

If we can rule that out and this starts significantly affecting us, then yeah, an infra ticket is probably warranted 👍

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants