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

ASF GitHub Bot commented on FLINK-6896:
---------------------------------------

Github user wuchong commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4111#discussion_r122141845
  
    --- Diff: 
flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/CodeGenerator.scala
 ---
    @@ -871,12 +871,24 @@ class CodeGenerator(
           returnType: TypeInformation[_ <: Any],
           resultFieldNames: Seq[String])
         : GeneratedExpression = {
    -    val input1AccessExprs = for (i <- 0 until input1.getArity if 
input1Mapping.contains(i))
    -      yield generateInputAccess(input1, input1Term, i, input1Mapping)
    +
    +    val input1AccessExprs = if (input1.isInstanceOf[PojoTypeInfo[_]]) {
    +      for (i <- 0 until input1Mapping.length)
    +        yield generateInputAccess(input1, input1Term, i, input1Mapping)
    +    } else {
    +      for (i <- 0 until input1.getArity if input1Mapping.contains(i))
    +        yield generateInputAccess(input1, input1Term, i, input1Mapping)
    +    }
     
         val input2AccessExprs = input2 match {
    -      case Some(ti) => for (i <- 0 until ti.getArity if 
input2Mapping.contains(i))
    -        yield generateInputAccess(ti, input2Term, i, input2Mapping)
    +      case Some(ti) => if (input2.isInstanceOf[PojoTypeInfo[_]]) {
    --- End diff --
    
    `input2` should be `ti`,  here `input2` is an `Option` which can't be 
`PojoTypeInfo`.


> Creating a table from a POJO and use table sink to output fail
> --------------------------------------------------------------
>
>                 Key: FLINK-6896
>                 URL: https://issues.apache.org/jira/browse/FLINK-6896
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>    Affects Versions: 1.3.0
>            Reporter: Mark You
>            Assignee: sunjincheng
>         Attachments: debug.png
>
>
> Following example fails at sink, using debug mode to see the reason of 
> ArrayIndexOutOfBoundException is cause by the input type is Pojo type not Row?
> Sample:
> {code:title=TumblingWindow.java|borderStyle=solid}
> public class TumblingWindow {
>     public static void main(String[] args) throws Exception {
>         List<Content> data = new ArrayList<Content>();
>         data.add(new Content(1L, "Hi"));
>         data.add(new Content(2L, "Hallo"));
>         data.add(new Content(3L, "Hello"));
>         data.add(new Content(4L, "Hello"));
>         data.add(new Content(7L, "Hello"));
>         data.add(new Content(8L, "Hello world"));
>         data.add(new Content(16L, "Hello world"));
>         final StreamExecutionEnvironment env = 
> StreamExecutionEnvironment.getExecutionEnvironment();
>         env.setStreamTimeCharacteristic(TimeCharacteristic.EventTime);
>         DataStream<Content> stream = env.fromCollection(data);
>         DataStream<Content> stream2 = stream.assignTimestampsAndWatermarks(
>                 new 
> BoundedOutOfOrdernessTimestampExtractor<Content>(Time.milliseconds(1)) {
>                     /**
>                      * 
>                      */
>                     private static final long serialVersionUID = 
> 410512296011057717L;
>                     @Override
>                     public long extractTimestamp(Content element) {
>                         return element.getRecordTime();
>                     }
>                 });
>         final StreamTableEnvironment tableEnv = 
> TableEnvironment.getTableEnvironment(env);
>         Table table = tableEnv.fromDataStream(stream2, 
> "urlKey,uplink,downlink,httpGetMessageCount,httpPostMessageCount,statusCode,rowtime.rowtime");
>         Table windowTable = 
> table.window(Tumble.over("1.hours").on("rowtime").as("w")).groupBy("w, 
> urlKey")
>                 
> .select("w.start,urlKey,uplink.sum,downlink.sum,httpGetMessageCount.sum,httpPostMessageCount.sum
>  ");
>         //table.printSchema();
>         TableSink<Row> windowSink = new 
> CsvTableSink("/Users/mark/Documents/specific-website-code.csv", ",", 1,
>                 WriteMode.OVERWRITE);
>         windowTable.writeToSink(windowSink);
>         // tableEnv.toDataStream(windowTable, Row.class).print();
>         env.execute();
>     }
>     public static class Content implements Serializable {
>         /**
>          * 
>          */
>         private static final long serialVersionUID = 1429246948772430441L;
>         private String urlKey;
>         private long recordTime;
>         // private String recordTimeStr;
>         private long httpGetMessageCount;
>         private long httpPostMessageCount;
>         private long uplink;
>         private long downlink;
>         private long statusCode;
>         private long statusCodeCount;
>         public Content() {
>             super();
>         }
>         public Content(long recordTime, String urlKey) {
>             super();
>             this.recordTime = recordTime;
>             this.urlKey = urlKey;
>         }
>         public String getUrlKey() {
>             return urlKey;
>         }
>         public void setUrlKey(String urlKey) {
>             this.urlKey = urlKey;
>         }
>         public long getRecordTime() {
>             return recordTime;
>         }
>         public void setRecordTime(long recordTime) {
>             this.recordTime = recordTime;
>         }
>         public long getHttpGetMessageCount() {
>             return httpGetMessageCount;
>         }
>         public void setHttpGetMessageCount(long httpGetMessageCount) {
>             this.httpGetMessageCount = httpGetMessageCount;
>         }
>         public long getHttpPostMessageCount() {
>             return httpPostMessageCount;
>         }
>         public void setHttpPostMessageCount(long httpPostMessageCount) {
>             this.httpPostMessageCount = httpPostMessageCount;
>         }
>         public long getUplink() {
>             return uplink;
>         }
>         public void setUplink(long uplink) {
>             this.uplink = uplink;
>         }
>         public long getDownlink() {
>             return downlink;
>         }
>         public void setDownlink(long downlink) {
>             this.downlink = downlink;
>         }
>         public long getStatusCode() {
>             return statusCode;
>         }
>         public void setStatusCode(long statusCode) {
>             this.statusCode = statusCode;
>         }
>         public long getStatusCodeCount() {
>             return statusCodeCount;
>         }
>         public void setStatusCodeCount(long statusCodeCount) {
>             this.statusCodeCount = statusCodeCount;
>         }
>     }
>     private class TimestampWithEqualWatermark implements 
> AssignerWithPunctuatedWatermarks<Object[]> {
>         /**
>          * 
>          */
>         private static final long serialVersionUID = 1L;
>         @Override
>         public long extractTimestamp(Object[] element, long 
> previousElementTimestamp) {
>             // TODO Auto-generated method stub
>             return (long) element[0];
>         }
>         @Override
>         public Watermark checkAndGetNextWatermark(Object[] lastElement, long 
> extractedTimestamp) {
>             return new Watermark(extractedTimestamp);
>         }
>     }
> }
> {code}
> Exception trace
> {code}
> Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 6
>       at 
> org.apache.flink.table.codegen.CodeGenerator.org$apache$flink$table$codegen$CodeGenerator$$generateFieldAccess(CodeGenerator.scala:1661)
>       at 
> org.apache.flink.table.codegen.CodeGenerator.org$apache$flink$table$codegen$CodeGenerator$$generateInputAccess(CodeGenerator.scala:1599)
>       at 
> org.apache.flink.table.codegen.CodeGenerator$$anonfun$26.apply(CodeGenerator.scala:875)
>       at 
> org.apache.flink.table.codegen.CodeGenerator$$anonfun$26.apply(CodeGenerator.scala:874)
>       at 
> scala.collection.TraversableLike$WithFilter$$anonfun$map$2.apply(TraversableLike.scala:728)
>       at scala.collection.immutable.Range.foreach(Range.scala:166)
>       at 
> scala.collection.TraversableLike$WithFilter.map(TraversableLike.scala:727)
>       at 
> org.apache.flink.table.codegen.CodeGenerator.generateConverterResultExpression(CodeGenerator.scala:874)
>       at 
> org.apache.flink.table.plan.nodes.CommonScan$class.generatedConversionFunction(CommonScan.scala:57)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamScan.generatedConversionFunction(DataStreamScan.scala:36)
>       at 
> org.apache.flink.table.plan.nodes.datastream.StreamScan$class.convertToInternalRow(StreamScan.scala:48)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamScan.convertToInternalRow(DataStreamScan.scala:36)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamScan.translateToPlan(DataStreamScan.scala:63)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamCalc.translateToPlan(DataStreamCalc.scala:94)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamGroupWindowAggregate.translateToPlan(DataStreamGroupWindowAggregate.scala:119)
>       at 
> org.apache.flink.table.plan.nodes.datastream.DataStreamCalc.translateToPlan(DataStreamCalc.scala:94)
>       at 
> org.apache.flink.table.api.StreamTableEnvironment.translateToCRow(StreamTableEnvironment.scala:678)
>       at 
> org.apache.flink.table.api.StreamTableEnvironment.translate(StreamTableEnvironment.scala:637)
>       at 
> org.apache.flink.table.api.StreamTableEnvironment.writeToSink(StreamTableEnvironment.scala:214)
>       at org.apache.flink.table.api.Table.writeToSink(table.scala:800)
>       at org.apache.flink.table.api.Table.writeToSink(table.scala:773)
>       at com.taiwanmobile.cep.noc.TumblingWindow.main(TumblingWindow.java:66)
> {code}



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

Reply via email to