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.

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

Separate dynein configuration from table schema cache #1

Merged
merged 2 commits into from
Nov 17, 2020

Conversation

thash
Copy link
Contributor

@thash thash commented Nov 17, 2020

Issue #, if available: N/A

Description of changes:

Before this PR dynein saves detail of the currently using table in ~/.dynein/config.yml file.
Now, config.yml only contains using_region and using_table to identify the target table, and a new file named cache.yml would store details of tables you have used so far.

$ cat ~/.dynein/config.yml
---
using_region: ap-northeast-1
using_table: books

$ cat ~/.dynein/cache.yml
---
tables:
  us-east-1/Employee:
    region: us-east-1
    name: Employee
    pk:
      name: LoginAlias
      kind: S
    sk: ~
    indexes: ~
    mode: OnDemand
  ap-northeast-1/books:
    region: ap-northeast-1
    name: books
    pk:
      name: isbn
      kind: S
    sk: ~
    indexes: ~
    mode: Provisioned

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@thash thash merged commit 961807b into awslabs:main Nov 17, 2020
@thash thash deleted the config_cache branch November 17, 2020 14:42
@thash thash mentioned this pull request Nov 18, 2020
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.

1 participant