Skip to content

Conversation

@vijaygupta18
Copy link
Member

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates

Description

Additional Changes

  • This PR modifies the database schema (database migration added)
  • This PR modifies dhall configs/environment variables

Motivation and Context

How did you test it?

Checklist

  • I formatted the code and addressed linter errors ./dev/format-all-files.sh
  • I reviewed submitted code
  • I added unit tests for my changes where possible
  • I added a CHANGELOG entry if applicable

@khuzema786 khuzema786 force-pushed the backend/kv-configs-reverse branch from 98e80cb to a121d1d Compare November 18, 2024 09:14
@khuzema786 khuzema786 force-pushed the backend/kv-configs-reverse branch from a121d1d to 77c0595 Compare November 19, 2024 05:38
kvTablesTtl :: HM.HashMap Text Integer,
useCAC :: [Text],
useCACForFrontend :: Bool,
readFromMasterDb :: [Text]
Copy link
Contributor

@khuzema786 khuzema786 Nov 19, 2024

Choose a reason for hiding this comment

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

{
  "enableKVForRead": ["table1", "table2", "table3", ...],
  "enableKVForWriteAlso": [{ "nameOfTable" : "table1", "percentEnable" : 100 }, ...],
  "disableForKV": ["table1", "table2", "table3", ...],
  "kvTablesTtl": {
    "table1": 3600,
    "table2": 7200,
    "table3": 14400,
     ...
  },
  "useCAC": ["table4", "table5", ...],
  "useCACForFrontend": true,
  "readFromMasterDb": ["table6", "table7", ...]
}

@khuzema786 khuzema786 merged commit 81711d8 into main Nov 19, 2024
1 check passed
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.

4 participants