You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hop.apache.org by "Adalennis (via GitHub)" <gi...@apache.org> on 2026/10/16 18:51:46 UTC
[PR] Add Beam Hive Catalog Input to the navigation and replace the hand-maintained list on the transforms page. #8977 (hop)
Adalennis opened a new pull request, #8978:
URL: https://github.com/apache/hop/pull/8978
Fixes #8977
- Adds Beam Hive catalog input to the navigation, in its alphabetical place among the Beam transforms. It was the only transform page missing from it; like SFTP Put, the other experimental transform, it is listed without an "(Experimental)" mark, since its page already warns about it.
- Replaces the hand-maintained list on the transforms overview page (`pipeline/transforms.adoc`), which missed 28 transforms, with a short text pointing to the navigation, the site search and the Help button of each transform's dialog, as the actions overview page does. Nothing includes that list. The rest of the page is kept, with two typos fixed ("HOP", "pay attention too").
- [x] Run `mvn clean install apache-rat:check` to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
- [x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using `git rebase -i`.
- [x] Mention the appropriate issue in your description (for example: `addresses #123`), if applicable.
To make clear that you license your contribution under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
you have to acknowledge this by using the following check-box.
- [x] I hereby declare this contribution to be licensed under the [Apache License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
- [ ] In any other case, please file an [Apache Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add Beam Hive Catalog Input to the navigation and replace the hand-maintained list on the transforms page. #8977 (hop)
Posted by "bamaer (via GitHub)" <gi...@apache.org>.
bamaer commented on PR #8978:
URL: https://github.com/apache/hop/pull/8978#issuecomment-6230860143
Thanks @Adalennis, good catch on the missing navigation entry, and keeping the list in one place makes sense.
Docs only, so nothing here can break anything. One thing worth fixing before merge:
**Worth fixing**
- Four pages still link here as a full list of transforms, and that list is gone after this PR:
- `getting-started/hop-next-steps.adoc:32` "the complete list of transforms"
- `pipeline/create-pipeline.adoc:82` "Check the list of transforms"
- `pipeline/pipelines.adoc:176` "Transforms Catalog — Browse the complete library of over 100 transforms."
- `snippets/hop-concepts/transform.adoc:21` "See the Transforms catalog for all available transforms."
Suggested fix: reword these so they no longer promise a list. For example, point them to "the transform pages under Pipelines › Transforms", or just link to "Transforms".
Checked against the source tree, site not built: the 28 missing transforms, that Beam Hive catalog input was the only transform page missing from the navigation, that nothing includes the removed list, and that the PR merges cleanly with main.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add Beam Hive Catalog Input to the navigation and replace the hand-maintained list on the transforms page. #8977 (hop)
Posted by "Adalennis (via GitHub)" <gi...@apache.org>.
Adalennis commented on PR #8978:
URL: https://github.com/apache/hop/pull/8978#issuecomment-6231270657
Thanks @bamaer, good catch. Reworded all four so they point to the transform pages in the navigation instead of a list.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add Beam Hive Catalog Input to the navigation and replace the hand-maintained list on the transforms page. #8977 (hop)
Posted by "bamaer (via GitHub)" <gi...@apache.org>.
bamaer merged PR #8978:
URL: https://github.com/apache/hop/pull/8978
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Add Beam Hive Catalog Input to the navigation and replace the hand-maintained list on the transforms page. #8977 (hop)
Posted by "bamaer (via GitHub)" <gi...@apache.org>.
bamaer commented on PR #8978:
URL: https://github.com/apache/hop/pull/8978#issuecomment-6231204519
The list in my previous comment is resolved in #8978.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@hop.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org