ruanwenjun commented on code in PR #1758: URL: https://github.com/apache/incubator-seatunnel/pull/1758#discussion_r859740108
########## seatunnel-connectors/seatunnel-connectors-spark/seatunnel-connector-spark-webhook/src/main/scala/org/apache/seatunnel/spark/webhook/source/HttpData.scala: ########## @@ -17,12 +17,12 @@ package org.apache.seatunnel.spark.webhook.source -import java.util.Date +import java.sql.Timestamp /** * Streaming data read from local server will have this schema * * @param value - The payload POSTed to http endpoint. * @param timestamp - Timestamp of when it was put on a stream. */ -case class HttpData(value: String, timestamp: Date) +case class HttpData(value: String, timestamp: Timestamp) Review Comment: I think use `long` type it better, there is no need to transform to `Timestamp` here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
