[ 
https://issues.apache.org/jira/browse/HADOOP-14068?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Mackrory updated HADOOP-14068:
-----------------------------------
    Attachment: HADOOP-14068-HADOOP-13345.003.patch

{quote}
I assume this change is because the table moves out of the Active state after 
changing throughput provisioning? So we wait for it to become Active before 
proceeding? And I assume we never hit the issue with the local dynamoDB?
{quote}

That's correct - it appears instantaneous locally.

{quote}
Don't think we want to swallow this exception. How about resetting the thread's 
interrupt flag then throwing an InterruptedIOException that wraps the original?
{quote}

Ah yes - will do.

{quote}
Just remove the line.. 
{quote}

Removed both sections of commented out code...

{quote}
For testing, it may just be best to switch to the real DDB endpoint everywhere
{quote}

I don't disagree about the benefits, but having TestDynamoDBMetadataStore be 
able to run in pre-commit checks seems like a pretty big deal to me. The full 
integration test version does take a long time to run (~7 minutes for me).


> Add integration test version of TestMetadataStore for DynamoDB
> --------------------------------------------------------------
>
>                 Key: HADOOP-14068
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14068
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/s3
>            Reporter: Sean Mackrory
>            Assignee: Sean Mackrory
>         Attachments: HADOOP-14068-HADOOP-13345.001.patch, 
> HADOOP-14068-HADOOP-13345.002.patch, HADOOP-14068-HADOOP-13345.003.patch
>
>
> I tweaked TestDynamoDBMetadataStore to run against the actual Amazon DynamoDB 
> service (as opposed to the "local" edition). Several tests failed because of 
> minor variations in behavior. I think the differences that are clearly 
> possible are enough to warrant extending that class as an ITest (but 
> obviously keeping the existing test so 99% of the the coverage remains even 
> when not configured for actual DynamoDB usage).



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to