[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-26 Thread Scott Carey (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12805310#action_12805310 ] Scott Carey commented on AVRO-380: -- I will work on this patch later this week. However, I c

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Philip Zeyliger (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806025#action_12806025 ] Philip Zeyliger commented on AVRO-380: -- bq. setSyncInterval() If you set syncInterval t

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Philip Zeyliger (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806026#action_12806026 ] Philip Zeyliger commented on AVRO-380: -- Meant to add: overall approach looks good. > Av

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806039#action_12806039 ] Doug Cutting commented on AVRO-380: --- I would prefer we use longs instead of ints for both t

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Scott Carey (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806067#action_12806067 ] Scott Carey commented on AVRO-380: -- bq. If you set syncInterval to Integer.MAX_VALUE, I bet

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Scott Carey (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806072#action_12806072 ] Scott Carey commented on AVRO-380: -- bq. I would prefer we use longs instead of ints for both

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-28 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806078#action_12806078 ] Doug Cutting commented on AVRO-380: --- I am okay with the implementation not working for bloc

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-29 Thread Philip Zeyliger (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806414#action_12806414 ] Philip Zeyliger commented on AVRO-380: -- +1. I'll commit this tomorrow, unless there's m

[jira] Commented: (AVRO-380) Avro Container File format change: add block size to block descriptor

2010-01-29 Thread Doug Cutting (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-380?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806425#action_12806425 ] Doug Cutting commented on AVRO-380: --- +1 to the patch. > we may be able to have a Decoder.i