Skip to content

Fix build failures by remove junit deps in presto-parquet module#17334

Merged
zhenxiao merged 1 commit intoprestodb:masterfrom
beinan:fix_build_parquet_junit
Feb 23, 2022
Merged

Fix build failures by remove junit deps in presto-parquet module#17334
zhenxiao merged 1 commit intoprestodb:masterfrom
beinan:fix_build_parquet_junit

Conversation

@beinan
Copy link
Copy Markdown
Member

@beinan beinan commented Feb 22, 2022

Using testng instead of junit to fix the error below in FB's build

[ERROR] Failed to execute goal on project presto-parquet: Could not resolve dependencies for project com.facebook.presto:presto-parquet:jar:0.272-SNAPSHOT: Failed to collect dependencies at junit:junit:jar:4.13.1: Failed to read artifact descriptor for junit:junit:jar:4.13.1: Could not transfer artifact junit:junit:pom:4.13.1 from/to nexus (https://maven.thefacebook.com/nexus/content/groups/public): Transfer failed for https://maven.thefacebook.com/nexus/content/groups/public/junit/junit/4.13.1/junit-4.13.1.pom: No route to host (Host unreachable) -> [Help 1]

== NO RELEASE NOTE ==

@beinan beinan force-pushed the fix_build_parquet_junit branch 3 times, most recently from 1d77580 to 3c3b900 Compare February 22, 2022 23:02
@beinan beinan changed the title Fix build failures by remove the real version of junit in presto-parquet module Fix build failures by remove junit deps in presto-parquet module Feb 22, 2022
@beinan beinan requested review from aweisberg and zhenxiao February 22, 2022 23:03
@beinan beinan force-pushed the fix_build_parquet_junit branch from 3c3b900 to a531566 Compare February 22, 2022 23:37
Copy link
Copy Markdown
Contributor

@aweisberg aweisberg left a comment

Choose a reason for hiding this comment

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

Thanks!

@shangxinli
Copy link
Copy Markdown
Collaborator

LGTM

@zhenxiao zhenxiao merged commit 39b6cea into prestodb:master Feb 23, 2022
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.

4 participants