accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Josh Elser (JIRA)" <>
Subject [jira] [Commented] (ACCUMULO-4141) RFile metadata not loading from cache
Date Mon, 22 Feb 2016 22:09:18 GMT


Josh Elser commented on ACCUMULO-4141:

A bit late since this was already committed (but maybe we can clean up in a follow-on):


Rely on {{StandardCharsets}} instead (and do the old Charset.forName in 1.6)? I feel like
avoiding Guava seeping in will be better in the long-run.

 The patch doesn't appear to include any new or modified tests. Please justify why no new
tests are needed for this patch. Also please list what manual steps were performed to verify
this patch. 

While I don't mean to assert that HadoopQA is binding, I have no understanding about whether
or not your changes are actually doing what you say they are. Any opinions on how to better
test this going forward? You had mentioned in IRC about making a real API around RFile, would
things get implicitly easier to test with such an API?

> RFile metadata not loading from cache 
> --------------------------------------
>                 Key: ACCUMULO-4141
>                 URL:
>             Project: Accumulo
>          Issue Type: Bug
>          Components: tserver
>    Affects Versions: 1.6.4, 1.7.0
>            Reporter: Keith Turner
>            Assignee: Keith Turner
>             Fix For: 1.6.6, 1.7.1, 1.8.0
>         Attachments: ACCUMULO-4141-1.6.patch
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
> In [BCFile line 818|]
an attempt to read crypto metadata from cache is made.  Then at [BCFile line 820|]
if that crypto metadata does not exist, some BC file metadata is read from the file.  For
unencrypted files this data will never exist in cache.  Therefore for unencrypted files metadata
will always be read from the file, even when its cached.

This message was sent by Atlassian JIRA

View raw message