[
https://issues.apache.org/jira/browse/ARROW-1663?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16222791#comment-16222791
]
ASF GitHub Bot commented on ARROW-1663:
---------------------------------------
BryanCutler commented on a change in pull request #1193: ARROW-1663: use
consistent name for null and not-null in FixedSizeLis…
URL: https://github.com/apache/arrow/pull/1193#discussion_r147513746
##########
File path:
java/vector/src/test/java/org/apache/arrow/vector/pojo/TestConvert.java
##########
@@ -48,6 +55,73 @@
*/
public class TestConvert {
+ private static String badSchemaJson = "{\n" +
+ " \"fields\" : [ {\n" +
+ " \"nullable\" : true,\n" +
+ " \"type\" : {\n" +
+ " \"name\" : \"struct\"\n" +
+ " },\n" +
+ " \"children\" : [ {\n" +
+ " \"nullable\" : true,\n" +
+ " \"type\" : {\n" +
+ " \"name\" : \"list\"\n" +
+ " },\n" +
+ " \"children\" : [ {\n" +
+ " \"nullable\" : true,\n" +
+ " \"type\" : {\n" +
+ " \"name\" : \"null\"\n" +
+ " },\n" +
+ " \"children\" : [ ],\n" +
+ " \"typeLayout\" : {\n" +
+ " \"vectors\" : [ ]\n" +
+ " },\n" +
+ " \"name\" : \"[DEFAULT]\"\n" +
+ " } ],\n" +
+ " \"typeLayout\" : {\n" +
+ " \"vectors\" : [ {\n" +
+ " \"type\" : \"VALIDITY\",\n" +
+ " \"typeBitWidth\" : 1\n" +
+ " }, {\n" +
+ " \"type\" : \"OFFSET\",\n" +
+ " \"typeBitWidth\" : 32\n" +
+ " } ]\n" +
+ " },\n" +
+ " \"name\" : \"list\"\n" +
+ " }, {\n" +
+ " \"nullable\" : true,\n" +
+ " \"type\" : {\n" +
+ " \"name\" : \"fixedsizelist\",\n" +
+ " \"listSize\" : 5\n" +
+ " },\n" +
+ " \"children\" : [ {\n" +
+ " \"nullable\" : true,\n" +
+ " \"type\" : {\n" +
+ " \"name\" : \"null\"\n" +
+ " },\n" +
+ " \"children\" : [ ],\n" +
+ " \"typeLayout\" : {\n" +
+ " \"vectors\" : [ ]\n" +
+ " },\n" +
+ " \"name\" : \"$data$\"\n" +
+ " } ],\n" +
+ " \"typeLayout\" : {\n" +
+ " \"vectors\" : [ {\n" +
+ " \"type\" : \"VALIDITY\",\n" +
+ " \"typeBitWidth\" : 1\n" +
+ " } ]\n" +
+ " },\n" +
+ " \"name\" : \"fixedlist\"\n" +
+ " } ],\n" +
+ " \"typeLayout\" : {\n" +
+ " \"vectors\" : [ {\n" +
+ " \"type\" : \"VALIDITY\",\n" +
+ " \"typeBitWidth\" : 1\n" +
+ " } ]\n" +
+ " },\n" +
+ " \"name\" : \"a\"\n" +
+ " } ]\n" +
+ "}\n";
Review comment:
This is a large block that's very hard to read just to check the name of a
field. Is there a more compact way you can test this?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> [Java] Follow up on ARROW-1347 and make schema backward compatible
> ------------------------------------------------------------------
>
> Key: ARROW-1663
> URL: https://issues.apache.org/jira/browse/ARROW-1663
> Project: Apache Arrow
> Issue Type: Bug
> Components: Java - Vectors
> Reporter: Yuliya Feldman
> Assignee: Yuliya Feldman
> Labels: pull-request-available
> Fix For: 0.8.0
>
>
> ARROW-1347 covered ListVector to have name of the field $data$ instead of
> [DEFAULT]
> We left FixedSizeListVector behind.
> Another case is backward compatibility - if schema was created before
> ARROW-1347 was in place application may still suffer from side effects as it
> would not be updated based on new code.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
