Skip to content

Optimize Spanner queries#46

Merged
zmerlynn merged 1 commit intogoogleforgames:mainfrom
zmerlynn:optimize-spanner
Mar 18, 2024
Merged

Optimize Spanner queries#46
zmerlynn merged 1 commit intogoogleforgames:mainfrom
zmerlynn:optimize-spanner

Conversation

@zmerlynn
Copy link
Collaborator

Based on internal discussions, this changes the world data schema so that world data is duplicated into each entity. This prevents hot-spotting on world data keyspace, instead allowing a cleaner partition across the entities.

BREAKING CHANGE: Changes Spanner schema in backwards incompatible manner.

Based on internal discussions, this changes the world data schema so
that world data is duplicated into each entity. This prevents
hot-spotting on world data keyspace, instead allowing a cleaner
partition across the entities.

BREAKING CHANGE: Changes Spanner schema in backwards incompatible manner.
Copy link
Collaborator

@zaratsian zaratsian left a comment

Choose a reason for hiding this comment

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

Nice updates - LGTM

@zmerlynn zmerlynn merged commit a6f4c74 into googleforgames:main Mar 18, 2024
zmerlynn added a commit to zmerlynn/GenAI-quickstart that referenced this pull request Mar 26, 2024
* Adds a Gemma deployment, but it needs an API key. I would like to
add a secret, probably in common, that has to be edited before
deployment, so I'm not advertising it in the README yet.

* After googleforgames#46, secondhand knowledge was reenabled. Disable for now -
this needs more prompt work. I tried a couple of things to improve it
that are included in this PR but ultimately disabled it again.

* Update TGI to 1.4.4
zmerlynn added a commit to zmerlynn/GenAI-quickstart that referenced this pull request Mar 26, 2024
* Adds a Gemma deployment, but it needs an API key. I would like to
add a secret, probably in common, that has to be edited before
deployment, so I'm not advertising it in the README yet.

* After googleforgames#46, secondhand knowledge was reenabled. Disable for now -
this needs more prompt work. I tried a couple of things to improve it
that are included in this PR but ultimately disabled it again.

* Update TGI to 1.4.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants