[Python] Remove legacy property definition syntax from parquet module and test them

[asfimport]

Reporter: Krisztian Szucs / @kszucs
Assignee: Krisztian Szucs / @kszucs

PRs and other links:

• GitHub Pull Request #2187

_{Note: This issue was originally created as ARROW-2760. Please see the migration documentation for further details.}

[asfimport]

Antoine Pitrou / @pitrou:
Can you elaborate on what you call "legacy property definition syntax"?

[asfimport]

Krisztian Szucs / @kszucs:
See the second block https://cython.readthedocs.io/en/latest/src/userguide/extension_types.html#properties

cdef class Spam:

    property cheese:

        "A doc string can go here."

        def __get__(self):
            # This is called when the property is read.
            ...

        def __set__(self, value):
            # This is called when the property is written.
            ...

        def __del__(self):
            # This is called when the property is deleted.

syntax is deprecated.

[asfimport]

Wes McKinney / @wesm:
Moved this off 0.10. If it gets merged in time, that's great, but doesn't fix bugs / add features

[asfimport]

Wes McKinney / @wesm:
Issue resolved by pull request 2187
#2187