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

ARROW-12806: [Python] test_write_to_dataset_filesystem missing a dataset mark #10346

Closed

Conversation

jorisvandenbossche
Copy link
Member

No description provided.

@github-actions
Copy link

@jorisvandenbossche
Copy link
Member Author

This begs the question if we should have a "minimal build" for python as well that turns the tests, otherwise we are to keep forget adding such markers (although in this case if that minimal build didn't include pandas, we still wouldn't have watched that error)

@lidavidm
Copy link
Member

I think it'd make sense for a minimal build to still include Pandas, since it doesn't change anything about the build itself, and would enable more tests, right?

@kszucs
Copy link
Member

kszucs commented May 17, 2021

Sounds like a good idea to me.

Copy link
Member

@lidavidm lidavidm left a comment

Choose a reason for hiding this comment

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

LGTM. Looks like Homebrew is having some trouble today.

@lidavidm lidavidm closed this in b38ab81 May 17, 2021
@lidavidm
Copy link
Member

Should this also get cherry-picked into 4.0.1?

@kszucs
Copy link
Member

kszucs commented May 17, 2021

While it's not essential, it shouldn't cause any trouble. Added to 4.0.1 jira version.

@jorisvandenbossche jorisvandenbossche deleted the ARROW-12806 branch May 17, 2021 15:48
@jorisvandenbossche
Copy link
Member Author

I think it'd make sense for a minimal build to still include Pandas, since it doesn't change anything about the build itself, and would enable more tests, right?

Yes, indeed, that doesn't impact the build. Having no pandas installed is also a case to test, but we actually do have a "no-pandas" build, so that is already covered.

kszucs pushed a commit that referenced this pull request May 17, 2021
…set mark

Closes #10346 from jorisvandenbossche/ARROW-12806

Authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
michalursa pushed a commit to michalursa/arrow that referenced this pull request Jun 13, 2021
…set mark

Closes apache#10346 from jorisvandenbossche/ARROW-12806

Authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
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.

3 participants