You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Jeremy Hinegardner (JIRA)" <ji...@apache.org> on 2011/01/03 06:13:45 UTC

[jira] Commented: (AVRO-724) C implementation does not write datum values that are larger than the memory write buffer (currently 16K)

    [ https://issues.apache.org/jira/browse/AVRO-724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12976614#action_12976614 ] 

Jeremy Hinegardner commented on AVRO-724:
-----------------------------------------

I am going to attempt to work on this issue.  If anyone has some guidance on the approach for this, I could use it.  Otherwise I'll figure something out.

> C implementation does not write datum values that are larger than the memory write buffer (currently 16K)
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: AVRO-724
>                 URL: https://issues.apache.org/jira/browse/AVRO-724
>             Project: Avro
>          Issue Type: Bug
>          Components: c
>    Affects Versions: 1.4.1
>            Reporter: Jeremy Hinegardner
>
> The current C implementation does not allow for datum values greater than 16K.
> The {{avro_file_writer_append}} flushes blocks to disk over time, but does not deal with the single case of a single datum being larger than {{avro_file_writer_t.datum_buffer}}.  This is noted in the source code:
> {code:title=datafile.c:294-313}
> int avro_file_writer_append(avro_file_writer_t w, avro_datum_t datum)
> {
>     int rval;                  
>     if (!w || !datum) {        
>         return EINVAL;         
>     }
>     rval = avro_write_data(w->datum_writer, w->writers_schema, datum);
>     if (rval) {                
>         check(rval, file_write_block(w));
>         rval =
>             avro_write_data(w->datum_writer, w->writers_schema, datum);
>         if (rval) {            
>             /* TODO: if the datum encoder larger than our buffer,
>                just write a single large datum */
>             return rval;       
>         }
>     }
>     w->block_count++;          
>     return 0;                  
> }
> {code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.