Revert fastparquet nullable dtype support

[lithomas1]

• closes #xxxx
• tests added / passed
• Ensure all linting tests pass, see here for how to run them
• whatsnew entry

[pep8speaks]

Hello @lithomas1! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2021-08-10 18:43:10 UTC

[lithomas1]

OK, failing tests are flaky. So this is ready for review now. The backport will be a bit tricky, so I'll do it manually. (Basically backport the last PR and this PR together. We're still gonna backport that one since it unpins fastparquet even though most of its changes are reverted here).

cc @jorisvandenbossche (sorry for not pinging last time) @jreback

[jorisvandenbossche]

Thanks a lot, looks good to me

The backport will be a bit tricky, so I'll do it manually. (Basically backport the last PR and this PR together. We're still gonna backport that one since it unpins fastparquet even though most of its changes are reverted here).

Another option is to first do a simple plain revert of the previous PR, and then include the changes of both PRs here (that will also make a easier to understand diff of the final changes), and then there is only a single PR to backport.

[jreback]

i am fine with this change.

[lithomas1]

ping for merge @jreback. Just merged master

[jreback]

thanks @lithomas1

[lumberbot-app]

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

1. Checkout backport branch and update it.

$ git checkout 1.3.x
$ git pull

2. Cherry pick the first parent branch of the this PR on top of the older branch:

$ git cherry-pick -m1 45fd72aa1768d2132ab0c294f1cb208d666a8965

3. You will likely have some merge/cherry-pick conflict here, fix them and commit:

$ git commit -am 'Backport PR #42954: Revert fastparquet nullable dtype support'

4. Push to a named branch :

git push YOURFORK 1.3.x:auto-backport-of-pr-42954-on-1.3.x

5. Create a PR against branch 1.3.x, I would have named this PR:

"Backport PR #42954 on branch 1.3.x"

And apply the correct labels and milestones.

Congratulation you did some good work ! Hopefully your backport PR will be tested by the continuous integration and merged soon!

If these instruction are inaccurate, feel free to suggest an improvement.

[jreback]

@meeseeksdev backport 1.3.x

[lumberbot-app]

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

1. Checkout backport branch and update it.

$ git checkout 1.3.x
$ git pull

2. Cherry pick the first parent branch of the this PR on top of the older branch:

$ git cherry-pick -m1 45fd72aa1768d2132ab0c294f1cb208d666a8965

3. You will likely have some merge/cherry-pick conflict here, fix them and commit:

$ git commit -am 'Backport PR #42954: Revert fastparquet nullable dtype support'

4. Push to a named branch :

git push YOURFORK 1.3.x:auto-backport-of-pr-42954-on-1.3.x

5. Create a PR against branch 1.3.x, I would have named this PR:

"Backport PR #42954 on branch 1.3.x"

And apply the correct labels and milestones.

Congratulation you did some good work ! Hopefully your backport PR will be tested by the continuous integration and merged soon!

If these instruction are inaccurate, feel free to suggest an improvement.

[jreback]

@lithomas1 if you wouldn't mind do the manual backport here

[simonjayhawkins]

Basically backport the last PR and this PR together.

The net result is no release note. do we need one? It's not clear to me what is changed/fixed?

[jorisvandenbossche]

Indeed, no release note, but the two combined PRs still have some changes to support the latest fastparquet version properly (in theory that could have been added as release note).

[simonjayhawkins]

@lithomas1 if you wouldn't mind do the manual backport here

@lithomas1 what is added and then removed in the release notes is not identical, so have synced the notes in #42985, so no need to backport that change.

[lithomas1]

Sorry, thanks for fixing that, planning on doing the backport now.