docs: Update DynamoDB BatchGetItem permissions

[TremaMiguel]

Signed-off-by: Miguel Trejo [email protected]

What this PR does / why we need it:

DynamoDBOnlineStore retrieve items calling the BatchGetItem operation, documentation is updated for this.

[codecov-commenter]

Codecov Report

Merging #2542 (5a3c573) into master (fbdd5fc) will decrease coverage by 24.04%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #2542       +/-   ##
===========================================
- Coverage   81.68%   57.63%   -24.05%     
===========================================
  Files         141      140        -1     
  Lines       12000    11888      -112     
===========================================
- Hits         9802     6852     -2950     
- Misses       2198     5036     +2838     

Flag	Coverage Δ
integrationtests	?
unittests	57.63% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
.../integration/online_store/test_online_retrieval.py	16.84% <0.00%> (-80.00%)	⬇️
.../integration/online_store/test_universal_online.py	15.09% <0.00%> (-78.35%)	⬇️
...fline_store/test_universal_historical_retrieval.py	23.88% <0.00%> (-76.12%)	⬇️
sdk/python/tests/utils/online_read_write_test.py	19.35% <0.00%> (-74.20%)	⬇️
sdk/python/feast/wait.py	23.52% <0.00%> (-70.59%)	⬇️
...gration/registration/test_feature_service_apply.py	31.25% <0.00%> (-68.75%)	⬇️
sdk/python/feast/infra/online_stores/redis.py	28.39% <0.00%> (-67.91%)	⬇️
sdk/python/tests/integration/e2e/test_usage_e2e.py	33.87% <0.00%> (-66.13%)	⬇️
...dk/python/tests/integration/e2e/test_validation.py	33.92% <0.00%> (-66.08%)	⬇️
sdk/python/tests/data/data_creator.py	34.78% <0.00%> (-65.22%)	⬇️
... and 64 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fbdd5fc...5a3c573. Read the comment docs.

[achals]

Do we only use batchget now, and never get? Or are there instance where we still use get?

[TremaMiguel]

@achals thanks for reviewing this

yes, though if the user use the DynamoDBTable class and calls _get_dynamodb_resource he can use that resource to do any available actions from boto dynamodb resource, including getItem.

[adchia]

/lgtm

[feast-ci-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adchia, TremaMiguel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [adchia]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment