Skip to content

HIVE-28883: Remove obsolete documents from hive-site - #46

Merged
okumin merged 2 commits into
apache:mainfrom
okumin:HIVE-28883-remove
May 6, 2025
Merged

HIVE-28883: Remove obsolete documents from hive-site#46
okumin merged 2 commits into
apache:mainfrom
okumin:HIVE-28883-remove

Conversation

@okumin

@okumin okumin commented Apr 5, 2025

Copy link
Copy Markdown
Contributor

This PR would remove obsolete pages that explain how to use Confluence, discontinued infrastructure or deprecated features.


# Apache Hive : End2EndTests

As discussed in HIVE-2985 (<http://issues.apache.org/jira/browse/HIVE-2985>), the goal of this exercise is to create a end to end test framework.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm considering voiding this ticket because we have not seen any progress for 12 years.
https://issues.apache.org/jira/browse/HIVE-2985

@zabetak zabetak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, just minor comments that we don't really need to handle as part of this PR!

Comment thread content/docs/latest/home_27362069.md Outdated

Following are a few rules of thumb that should be followed when adding new test cases in Hive that require the introduction of new query file(s). Of course, these rules should not be applied if they invalidate the purpose of your test to begin with. These are generally helpful in keeping the test queries concise, minimizing the redundancies where possible, and ensuring that cascading failures due to a single test failure do not occur.

* Instead of creating your own data file for loading into a new table, use existing data from staged tables like `src`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: I like tests that are self-contained and minimal so I am not sure if using built-in datasets (especially src where the data are pretty cryptic) is always a good idea. Anyways this PR just moves around existing content so I am fine if it remains as is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm keeping this statement because we might have not achieved consensus.

Comment thread content/Development/qtest.md Outdated

@zabetak zabetak left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@okumin

okumin commented May 6, 2025

Copy link
Copy Markdown
Contributor Author

Thanks! I removed HiveWebInterface

@okumin
okumin merged commit f056ea6 into apache:main May 6, 2025
@okumin
okumin deleted the HIVE-28883-remove branch May 6, 2025 05:53
@okumin

okumin commented May 6, 2025

Copy link
Copy Markdown
Contributor Author

Merged. @zabetak Thanks for reviewing the pull request!

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.

2 participants