Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: googleapis/python-bigtable Loading
base: v2.11.2
Choose a base ref
...
head repository: googleapis/python-bigtable Loading
compare: v2.11.3
Choose a head ref
  • 4 commits
  • 8 files changed
  • 4 contributors

Commits on Aug 17, 2022

  1. chore(deps): update dependency google-cloud-monitoring to v2.11.1 (#635)

    @renovate-bot
    renovate-bot committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    05aae56 View commit details
    Browse the repository at this point in the history

  2. chore: move row value classes out of row_data (#633) 

    * chore: move row value classes out of row_data

This is in preparation for extracting row merging into a separate class. See #628

Co-authored-by: Anthonios Partheniou <[email protected]>
    @igorbernstein2 @parthea
    igorbernstein2 and parthea committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    e862460 View commit details
    Browse the repository at this point in the history

  3. perf: optimize row merging (#628) 

    This PR rewrites the row merging logic to be more correct and improve performance:
- extract row merging logic into its own class to simplify complexity of ReadRows handling
- Use OrderedDict instead of dict() for `{family: { qualifier: [] }}` data, this should maintain serverside ordering (family in creation order and qualifier in lexiographical). 
- define an explicit state machine with states implemented as methods
- add various optimizations like:
  - __slots__ on hot objects to avoid dict lookups
  - avoiding dict lookups for contiguous family and qualifier keys
 
Overall this improves performance by 20% and in my opinion is a lot more readable
    @igorbernstein2
    igorbernstein2 committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    c71ec70 View commit details
    Browse the repository at this point in the history

  4. chore(main): release 2.11.3 (#638) 

    Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
    @release-please
    release-please[bot] committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    bdc7929 View commit details
    Browse the repository at this point in the history
Loading