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.

Sign up for GitHub

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

Jump to bottom

GH-36954: [Python] Add more FlightInfo / FlightEndpoint attributes #43537

Merged
merged 32 commits into from
Nov 7, 2024
Merged

GH-36954: [Python] Add more FlightInfo / FlightEndpoint attributes #43537

merged 32 commits into from
Nov 7, 2024
+307 −42

Commits on Nov 7, 2024

  1. Add ordered and app_metadata attributes to PyArrow FlightInfo

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    47a3902 View commit details
    Browse the repository at this point in the history

  2. Fix code style

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    bfca244 View commit details
    Browse the repository at this point in the history

  3. Use same formatting in __repr__ as in cpp ToString

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    bd53f7f View commit details
    Browse the repository at this point in the history

  4. Allow for Python str app_metadata

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    5555bad View commit details
    Browse the repository at this point in the history

  5. Make ordered and app_metadata optional

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    1e87b19 View commit details
    Browse the repository at this point in the history

  6. Make total_records and total_bytes optional, support None

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    911e991 View commit details
    Browse the repository at this point in the history

  7. Add expiration_time and app_metadata attributes to PyArrow FlightEndp… 

    …oint
    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    9ab0624 View commit details
    Browse the repository at this point in the history

  8. Fix code style

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    e54872b View commit details
    Browse the repository at this point in the history

  9. Use microseconds precision to work with macOS

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    eb98843 View commit details
    Browse the repository at this point in the history

  10. Fix code style

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    2382b0e View commit details
    Browse the repository at this point in the history

  11. Add license to chrono.pxd

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    7501924 View commit details
    Browse the repository at this point in the history

  12. Allow for Python str app_metadata

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    4355408 View commit details
    Browse the repository at this point in the history

  13. Make expiration_time and app_metadata optional

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    4c09c13 View commit details
    Browse the repository at this point in the history

  14. Make CTimestampType timezone constructor arg const

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    60ec782 View commit details
    Browse the repository at this point in the history

  15. Remove .hex, use bool __repr__, use -1 not None default value

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    336b3e4 View commit details
    Browse the repository at this point in the history

  16. Support nanoseconds, cast expiration_time in FlightEndpoint() to time… 

    …_point::duration
    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    176347a View commit details
    Browse the repository at this point in the history

  17. Test FlightEndpoint with nanoseconds

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    29ad932 View commit details
    Browse the repository at this point in the history

  18. Add docstrings

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    d30bc6c View commit details
    Browse the repository at this point in the history

  19. Use None as default for total_records and total_bytes

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    428162d View commit details
    Browse the repository at this point in the history

  20. Simplify code with TimePoint_from_ns and TimePoint_to_ns

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    86e3fad View commit details
    Browse the repository at this point in the history

  21. Remove chrono.pxd, use existing CTimePoint instead

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    08b1a58 View commit details
    Browse the repository at this point in the history

  22. Fix Flight Timestamp definition to OS-agnostic nanoseconds

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    9dc0e75 View commit details
    Browse the repository at this point in the history

  23. Revert "Fix Flight Timestamp definition to OS-agnostic nanoseconds" 

    This reverts commit 84bbe5f.
    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    989f7a2 View commit details
    Browse the repository at this point in the history

  24. Fix conversion to and from system time

    @adamreeve @EnricoMi
    adamreeve authored and EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    08d6b48 View commit details
    Browse the repository at this point in the history

  25. Fix lint error

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    985b115 View commit details
    Browse the repository at this point in the history

  26. Fix docstring typo 

    Co-authored-by: David Li <li.davidm96@gmail.com>
    @EnricoMi @lidavidm
    EnricoMi and lidavidm committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    d264ce1 View commit details
    Browse the repository at this point in the history

  27. Emphasize app_metadata non-nullability in docstring

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    bf62c5d View commit details
    Browse the repository at this point in the history

  28. Apply suggestions from code review 

    Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
    @EnricoMi @jorisvandenbossche
    EnricoMi and jorisvandenbossche committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    818660f View commit details
    Browse the repository at this point in the history

  29. Apply review suggestion around CTimestampType with timezone 

    Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
    @EnricoMi @jorisvandenbossche
    EnricoMi and jorisvandenbossche committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    c88fdd5 View commit details
    Browse the repository at this point in the history

  30. Cleanup after committing suggestions

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    828070e View commit details
    Browse the repository at this point in the history

  31. Check argument types in FlightEndpoint()

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    d2d7cab View commit details
    Browse the repository at this point in the history

  32. Add comments on timestamp cast needs

    @EnricoMi
    EnricoMi committed Nov 7, 2024
    Configuration menu
    Copy the full SHA
    94a40b2 View commit details
    Browse the repository at this point in the history