Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[No-Merge] Test Hive_view with Spark #11

Closed
wants to merge 17 commits into from
Closed

Conversation

nk1506
Copy link
Owner

@nk1506 nk1506 commented Mar 7, 2024

No description provided.

@github-actions github-actions bot added the SPARK label Mar 7, 2024
@nk1506 nk1506 force-pushed the hive_view_with_spark branch from 924b2b9 to 9117989 Compare March 7, 2024 09:27
@github-actions github-actions bot added the HIVE label Mar 7, 2024
@@ -1318,8 +1320,8 @@ public void createViewWithSubqueryExpressionInFilterThatIsRewritten()
sql("USE spark_catalog");

assertThatThrownBy(() -> sql(sql))
.isInstanceOf(AnalysisException.class)
.hasMessageContaining(String.format("The table or view `%s` cannot be found", tableName));
.isInstanceOf(SparkException.class)
Copy link
Owner Author

Choose a reason for hiding this comment

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

In Case of Hive Catalog the thrown exception is type of SparkException

@@ -1430,29 +1432,29 @@ public void showViews() throws NoSuchTableException {
Object[] tempView = row("", "tempviewforlisting", true);
assertThat(sql("SHOW VIEWS"))
.contains(
row(NAMESPACE.toString(), "prefixV2", false),
row(NAMESPACE.toString(), "prefixV3", false),
row(NAMESPACE.toString(), "prefixv2", false),
Copy link
Owner Author

Choose a reason for hiding this comment

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

Hive is always returning it in lower case.

+ " 'provider' = 'iceberg')\n"
+ "AS\n%s\n",
catalogName, NAMESPACE, viewName, NAMESPACE, viewName, sql);
catalogName, NAMESPACE, viewName, TestHiveMetastore.HIVE_LOCAL_DIR, viewName, sql);
Copy link
Owner Author

Choose a reason for hiding this comment

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

Hive is initialising it's temp path with prefix hive.

@nk1506 nk1506 force-pushed the hive_view_new branch 7 times, most recently from a9c09c9 to 7a83702 Compare March 17, 2024 05:54
@nk1506 nk1506 force-pushed the hive_view_new branch 4 times, most recently from b3d4525 to 57eff75 Compare March 28, 2024 05:10
@nk1506 nk1506 force-pushed the hive_view_new branch 2 times, most recently from 7d44d05 to a188863 Compare March 29, 2024 09:24
nastra and others added 7 commits April 2, 2024 11:34
…e#10072)

Bumps software.amazon.awssdk:bom from 2.25.18 to 2.25.21.

---
updated-dependencies:
- dependency-name: software.amazon.awssdk:bom
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…he#9988)

Bumps org.glassfish.jaxb:jaxb-runtime from 2.3.3 to 2.3.9.

---
updated-dependencies:
- dependency-name: org.glassfish.jaxb:jaxb-runtime
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@nk1506 nk1506 force-pushed the hive_view_with_spark branch from 9117989 to 21b4255 Compare April 2, 2024 06:07
@nk1506 nk1506 changed the base branch from hive_view_new to main April 2, 2024 06:08
@nk1506 nk1506 closed this Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants