[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-16 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13873453#comment-13873453 ] Tie Liu commented on AVRO-1428: --- In following example, let's say we have unionSchema like

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-16 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13873831#comment-13873831 ] Tie Liu commented on AVRO-1428: --- yes, a given call to RecordSchema#computeHash keeps track of

[jira] [Updated] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-15 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tie Liu updated AVRO-1428: -- Attachment: TestAvro.java This is code my colleague to test the hash code generation. Schema.computeHash() to

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-15 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13872656#comment-13872656 ] Tie Liu commented on AVRO-1428: --- Our code doesn't compute hashcode directly, but our schema

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-15 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13872676#comment-13872676 ] Tie Liu commented on AVRO-1428: --- To explain the problem more clearly, in current

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-15 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13872703#comment-13872703 ] Tie Liu commented on AVRO-1428: --- it's half right that I suggest cache the field hashCode. In

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-15 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13872716#comment-13872716 ] Tie Liu commented on AVRO-1428: --- also for the UnionSchema, it's calculating the hashcode for

[jira] [Commented] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-09 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13866842#comment-13866842 ] Tie Liu commented on AVRO-1428: --- new patch file attached, it's generated using svn diff. The

[jira] [Created] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-06 Thread Tie Liu (JIRA)
Tie Liu created AVRO-1428: - Summary: Schema.computeHash() to add if check to avoid unnecessary hashcode computation Key: AVRO-1428 URL: https://issues.apache.org/jira/browse/AVRO-1428 Project: Avro

[jira] [Updated] (AVRO-1428) Schema.computeHash() to add if check to avoid unnecessary hashcode computation

2014-01-06 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tie Liu updated AVRO-1428: -- Attachment: AVRO-1428.patch This patch is to be applied on Schema.java under org.apache.avro package.

[jira] [Commented] (AVRO-1348) Improve Utf8 to String conversion

2013-12-16 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13849206#comment-13849206 ] Tie Liu commented on AVRO-1348: --- Another note about this is, change to use string literal

[jira] [Commented] (AVRO-1348) Improve Utf8 to String conversion

2013-12-16 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13849746#comment-13849746 ] Tie Liu commented on AVRO-1348: --- Just run the Perf test in our environment, which is 64 bit

[jira] [Created] (AVRO-1411) org.apache.avro.util.Utf8 performance improvement by remove private Charset in class

2013-12-12 Thread Tie Liu (JIRA)
Tie Liu created AVRO-1411: - Summary: org.apache.avro.util.Utf8 performance improvement by remove private Charset in class Key: AVRO-1411 URL: https://issues.apache.org/jira/browse/AVRO-1411 Project: Avro

[jira] [Commented] (AVRO-1411) org.apache.avro.util.Utf8 performance improvement by remove private Charset in class

2013-12-12 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13846578#comment-13846578 ] Tie Liu commented on AVRO-1411: --- I checked the patch, the patch v2

[jira] [Resolved] (AVRO-1411) org.apache.avro.util.Utf8 performance improvement by remove private Charset in class

2013-12-12 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tie Liu resolved AVRO-1411. --- Resolution: Duplicate duplicate as jira https://issues.apache.org/jira/browse/AVRO-1348

[jira] [Commented] (AVRO-1348) Improve Utf8 to String conversion

2013-12-12 Thread Tie Liu (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13846597#comment-13846597 ] Tie Liu commented on AVRO-1348: --- I tested it under java 6, will be out of office tomorrow,