Skip to content

Commit e887706

Browse files
committed
Update, array_element to array
1 parent 3d38a75 commit e887706

File tree

2 files changed

+19
-19
lines changed

2 files changed

+19
-19
lines changed

sql/core/src/main/scala/org/apache/spark/sql/parquet/CatalystSchemaConverter.scala

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -446,7 +446,7 @@ private[parquet] class CatalystSchemaConverter(
446446
field.name,
447447
Types
448448
.buildGroup(REPEATED)
449-
.addField(convertField(StructField("array_element", elementType, nullable)))
449+
.addField(convertField(StructField("array", elementType, nullable)))
450450
.named(CatalystConverter.ARRAY_CONTAINS_NULL_BAG_SCHEMA_NAME))
451451

452452
// Spark 1.4.x and prior versions convert ArrayType with non-nullable elements into a 2-level
@@ -459,7 +459,7 @@ private[parquet] class CatalystSchemaConverter(
459459
ConversionPatterns.listType(
460460
repetition,
461461
field.name,
462-
convertField(StructField("array_element", elementType, nullable), REPEATED))
462+
convertField(StructField("array", elementType, nullable), REPEATED))
463463

464464
// Spark 1.4.x and prior versions convert MapType into a 3-level group annotated by
465465
// MAP_KEY_VALUE. This is covered by `convertGroupField(field: GroupType): DataType`.
@@ -490,7 +490,7 @@ private[parquet] class CatalystSchemaConverter(
490490
.buildGroup(repetition).as(LIST)
491491
.addField(
492492
Types.repeatedGroup()
493-
.addField(convertField(StructField("array_element", elementType, containsNull)))
493+
.addField(convertField(StructField("array", elementType, containsNull)))
494494
.named("list"))
495495
.named(field.name)
496496

sql/core/src/test/scala/org/apache/spark/sql/parquet/ParquetSchemaSuite.scala

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
174174
"""
175175
|message root {
176176
| optional group _1 (LIST) {
177-
| repeated int32 array_element;
177+
| repeated int32 array;
178178
| }
179179
|}
180180
""".stripMargin)
@@ -185,7 +185,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
185185
|message root {
186186
| optional group _1 (LIST) {
187187
| repeated group list {
188-
| required int32 array_element;
188+
| required int32 array;
189189
| }
190190
| }
191191
|}
@@ -198,7 +198,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
198198
|message root {
199199
| optional group _1 (LIST) {
200200
| repeated group bag {
201-
| optional int32 array_element;
201+
| optional int32 array;
202202
| }
203203
| }
204204
|}
@@ -210,7 +210,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
210210
|message root {
211211
| optional group _1 (LIST) {
212212
| repeated group list {
213-
| optional int32 array_element;
213+
| optional int32 array;
214214
| }
215215
| }
216216
|}
@@ -267,7 +267,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
267267
| optional binary _1 (UTF8);
268268
| optional group _2 (LIST) {
269269
| repeated group bag {
270-
| optional group array_element {
270+
| optional group array {
271271
| required int32 _1;
272272
| required double _2;
273273
| }
@@ -290,7 +290,7 @@ class ParquetSchemaInferenceSuite extends ParquetSchemaTest {
290290
| optional binary _1 (UTF8);
291291
| optional group _2 (LIST) {
292292
| repeated group list {
293-
| optional group array_element {
293+
| optional group array {
294294
| required int32 _1;
295295
| required double _2;
296296
| }
@@ -467,7 +467,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
467467
"""message root {
468468
| optional group f1 (LIST) {
469469
| repeated group list {
470-
| optional int32 array_element;
470+
| optional int32 array;
471471
| }
472472
| }
473473
|}
@@ -482,7 +482,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
482482
nullable = true))),
483483
"""message root {
484484
| optional group f1 (LIST) {
485-
| repeated group array_element {
485+
| repeated group array {
486486
| optional int32 num;
487487
| }
488488
| }
@@ -496,7 +496,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
496496
"""message root {
497497
| optional group f1 (LIST) {
498498
| repeated group list {
499-
| required int32 array_element;
499+
| required int32 array;
500500
| }
501501
| }
502502
|}
@@ -508,7 +508,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
508508
StructField("f1", ArrayType(IntegerType, containsNull = false), nullable = true))),
509509
"""message root {
510510
| optional group f1 (LIST) {
511-
| repeated group array_element {
511+
| repeated group array {
512512
| required int32 num;
513513
| }
514514
| }
@@ -521,7 +521,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
521521
StructField("f1", ArrayType(IntegerType, containsNull = false), nullable = true))),
522522
"""message root {
523523
| optional group f1 (LIST) {
524-
| repeated int32 array_element;
524+
| repeated int32 array;
525525
| }
526526
|}
527527
""".stripMargin)
@@ -539,7 +539,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
539539
nullable = true))),
540540
"""message root {
541541
| optional group f1 (LIST) {
542-
| repeated group array_element {
542+
| repeated group array {
543543
| required binary str (UTF8);
544544
| required int32 num;
545545
| }
@@ -599,7 +599,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
599599
"""message root {
600600
| optional group f1 (LIST) {
601601
| repeated group list {
602-
| optional int32 array_element;
602+
| optional int32 array;
603603
| }
604604
| }
605605
|}
@@ -616,7 +616,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
616616
"""message root {
617617
| optional group f1 (LIST) {
618618
| repeated group bag {
619-
| optional int32 array_element;
619+
| optional int32 array;
620620
| }
621621
| }
622622
|}
@@ -632,7 +632,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
632632
"""message root {
633633
| optional group f1 (LIST) {
634634
| repeated group list {
635-
| required int32 array_element;
635+
| required int32 array;
636636
| }
637637
| }
638638
|}
@@ -648,7 +648,7 @@ class ParquetSchemaSuite extends ParquetSchemaTest {
648648
nullable = true))),
649649
"""message root {
650650
| optional group f1 (LIST) {
651-
| repeated int32 array_element;
651+
| repeated int32 array;
652652
| }
653653
|}
654654
""".stripMargin)

0 commit comments

Comments
 (0)