You are viewing a plain text version of this content. The canonical link for it is here.

Posted to user@accumulo.apache.org by Nabeel Hopeson <na...@gmail.com> on 2012/04/18 21:28:17 UTC

Documentation feedback

Hello,

There is some confusion on the following page
http://accumulo.apache.org/1.4/user_manual/Table_Design.html#Indexing.

In the section "Indexing", row ids are fetched by the following code

for(Entry<Key,Value> entry : indexScanner)

matchingRows.add(new Text(entry.getValue()));

According to the diagram within the Indexing section, I would have expected
this code

for(Entry<Key,Value> entry : indexScanner)

matchingRows.add(new Text(entry.getKey(). getColumnQualifier()));

Can you provide clarification in the documentation if I am misunderstanding
how it works? It is confusing that it would retrieve the "value" of the
entry when the row id is in the column qualifier.

Finally, the last line in that same example uses an undeclared variable e.

System.out.println(e.getValue());

but I think what was meant was

System.out.println(entry.getVa lue());

Hopefully my feedback will be of use to others,

Thanks,

Nabeel Hopeson

Re: Documentation feedback

Posted by Aaron Cordova <aa...@cordovas.org>.

I believe you're right about these being errors.

Thanks for the feedback!

On Apr 18, 2012, at 3:28 PM, Nabeel Hopeson wrote:

> Hello,
> 
> There is some confusion on the following page http://accumulo.apache.org/1.4/user_manual/Table_Design.html#Indexing.
> 
> In the section "Indexing", row ids are fetched by the following code
> 
> for(Entry<Key,Value> entry : indexScanner)
> 
> matchingRows.add(new Text(entry.getValue()));
> 
> According to the diagram within the Indexing section, I would have expected this code
> 
> for(Entry<Key,Value> entry : indexScanner)
> 
> matchingRows.add(new Text(entry.getKey(). getColumnQualifier()));
> 
> Can you provide clarification in the documentation if I am misunderstanding how it works? It is confusing that it would retrieve the "value" of the entry when the row id is in the column qualifier.
> 
> Finally, the last line in that same example uses an undeclared variable e.
> 
> System.out.println(e.getValue());
> 
> but I think what was meant was
> 
> System.out.println(entry.getVa lue());
> 
> Hopefully my feedback will be of use to others,
> 
> Thanks,
> 
> Nabeel Hopeson
>

Re: Documentation feedback

Posted by Billie J Rinaldi <bi...@ugov.gov>.

Thanks!  These will be fixed in ACCUMULO-546.  Let us know if you notice anything else that is incorrect.  :)

Billie


----- Original Message -----
> From: "Nabeel Hopeson" <na...@gmail.com>
> To: user@accumulo.apache.org
> Sent: Wednesday, April 18, 2012 3:28:17 PM
> Subject: Documentation feedback
> Hello,
> 
> There is some confusion on the following page
> http://accumulo.apache.org/1.4/user_manual/Table_Design.html#Indexing
> .
> 
> In the section "Indexing", row ids are fetched by the following code
> 
> for(Entry<Key,Value> entry : indexScanner)
> 
> matchingRows.add(new Text(entry.getValue()));
> 
> According to the diagram within the Indexing section, I would have
> expected this code
> 
> for(Entry<Key,Value> entry : indexScanner)
> 
> matchingRows.add(new Text(entry.getKey(). getColumnQualifier()));
> 
> Can you provide clarification in the documentation if I am
> misunderstanding how it works? It is confusing that it would retrieve
> the "value" of the entry when the row id is in the column qualifier.
> 
> Finally, the last line in that same example uses an undeclared
> variable e.
> 
> System.out.println(e.getValue());
> 
> but I think what was meant was
> 
> System.out.println(entry.getVa lue());
> 
> Hopefully my feedback will be of use to others,
> 
> Thanks,
> 
> Nabeel Hopeson