Skip to content

ARROW-7752: [Release] Enable and test dataset in the verification script - #6346

Closed
kszucs wants to merge 1 commit into
apache:masterfrom
kszucs:dataset-verification
Closed

ARROW-7752: [Release] Enable and test dataset in the verification script#6346
kszucs wants to merge 1 commit into
apache:masterfrom
kszucs:dataset-verification

Conversation

@kszucs

@kszucskszucs commented Feb 3, 2020

Copy link
Copy Markdown
Member

We're not testing the dataset feature in the verifications scripts yet.

@github-actions

Copy link
Copy Markdown

@xhochyxhochy 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.

+1, LGTM

Feel free to merge, I don't have the script at hand currently.

kou
kou approved these changes Feb 4, 2020

@koukou 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.

+1

@koukou closed this in 992d9fcFeb 4, 2020
kszucs added a commit that referenced this pull request Feb 7, 2020
We're not testing the dataset feature in the verifications scripts yet.
Closes#6346 from kszucs/dataset-verification and squashes the following commits:
b8530ea <Krisztián Szűcs> Test dataset during the verification
Authored-by: Krisztián Szűcs <szucs.krisztian@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for freeto 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.

3 participants

@kszucs@kou@xhochy