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

Scott Carey commented on AVRO-1282:
-----------------------------------

UnsafeUtils is a public class with public static content -- if we want to 
guarantee that it is only accessed in a controlled way, it will need to be 
hidden.


Additionally, the unit tests fail with this patch for me in avro-ipc:

{noformat}
Tests in error: 
  testError(org.apache.avro.TestNamespaceReflect): 
org.apache.avro.AvroRuntimeException: No field named message in: class 
org.apache.avro.test.errors.TestError
  testEcho(org.apache.avro.TestProtocolReflect): java.lang.NullPointerException
  testAdd(org.apache.avro.TestProtocolReflect): java.lang.NullPointerException
  testEchoBytes(org.apache.avro.TestProtocolReflect): 
java.lang.NullPointerException
  testEcho(org.apache.avro.TestProtocolReflectMeta): 
java.lang.NullPointerException
  testAdd(org.apache.avro.TestProtocolReflectMeta): 
java.lang.NullPointerException
  testEchoBytes(org.apache.avro.TestProtocolReflectMeta): 
java.lang.NullPointerException
{noformat}





                
> Make use of the sun.misc.Unsafe class during serialization if a JDK supports 
> it
> -------------------------------------------------------------------------------
>
>                 Key: AVRO-1282
>                 URL: https://issues.apache.org/jira/browse/AVRO-1282
>             Project: Avro
>          Issue Type: Improvement
>          Components: java
>    Affects Versions: 1.7.4
>            Reporter: Leo Romanoff
>            Priority: Minor
>         Attachments: avro-1282-v1.patch, avro-1282-v2.patch, 
> avro-1282-v3.patch, avro-1282-v4.patch, avro-1282-v5.patch, 
> avro-1282-v6.patch, avro-1282-v7.patch, avro-1282-v8.patch, 
> TestUnsafeUtil.java
>
>
> Unsafe can be used to significantly speed up serialization process, if a JDK 
> implementation supports java.misc.Unsafe properly. Most JDKs running on PCs 
> support it. Some platforms like Android lack a proper support for Unsafe yet.
> There are two possibilities to use Unsafe for serialization:
> 1) Very quick access to the fields of objects. It is way faster than with the 
> reflection-based approach using Field.get/set
> 2) Input and Output streams can be using Unsafe to perform very quick 
> input/output.
>  
> 3) More over, Unsafe makes it possible to serialize to/deserialize from 
> off-heap memory directly and very quickly, without any intermediate buffers 
> allocated on heap. There is virtually no overhead compared to the usual byte 
> arrays.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to