[jira] [Commented] (ARROW-1431) [Java] JsonFileReader doesn't intialize some vectors approperately

2019-04-17 Thread Li Jin (JIRA)


[ 
https://issues.apache.org/jira/browse/ARROW-1431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16820204#comment-16820204
 ] 

Li Jin commented on ARROW-1431:
---

This issue is fixed by https://github.com/apache/arrow/pull/1290

> [Java] JsonFileReader doesn't intialize some vectors approperately 
> ---
>
> Key: ARROW-1431
> URL: https://issues.apache.org/jira/browse/ARROW-1431
> Project: Apache Arrow
>  Issue Type: Bug
>  Components: Java
>Reporter: Li Jin
>Priority: Major
>
> One example is for ListVector, the JsonFileReader sets the validity, offset 
> and data, but doesn't set `lastSet` variable in the ListVector instance.
> ArrowFileReader works correct before it invokes `loadFieldBuffers` and 
> intialize `lastSet` correctly:
> https://github.com/apache/arrow/blob/master/java/vector/src/main/java/org/apache/arrow/vector/complex/ListVector.java#L120
> This doesn't break integration tests but can cause weird bugs when people 
> call methods on the vector read from json.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (ARROW-1431) [Java] JsonFileReader doesn't intialize some vectors approperately

2017-11-07 Thread Li Jin (JIRA)

[ 
https://issues.apache.org/jira/browse/ARROW-1431?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16242664#comment-16242664
 ] 

Li Jin commented on ARROW-1431:
---

https://github.com/apache/arrow/pull/1290

> [Java] JsonFileReader doesn't intialize some vectors approperately 
> ---
>
> Key: ARROW-1431
> URL: https://issues.apache.org/jira/browse/ARROW-1431
> Project: Apache Arrow
>  Issue Type: Bug
>Reporter: Li Jin
>
> One example is for ListVector, the JsonFileReader sets the validity, offset 
> and data, but doesn't set `lastSet` variable in the ListVector instance.
> ArrowFileReader works correct before it invokes `loadFieldBuffers` and 
> intialize `lastSet` correctly:
> https://github.com/apache/arrow/blob/master/java/vector/src/main/java/org/apache/arrow/vector/complex/ListVector.java#L120
> This doesn't break integration tests but can cause weird bugs when people 
> call methods on the vector read from json.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)