You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/07/21 11:14:27 UTC

[GitHub] [flink] zentol opened a new pull request, #20332: [FLINK-28624][csv] Accept mapper/schema factories

zentol opened a new pull request, #20332:
URL: https://github.com/apache/flink/pull/20332

   Adjusts the CsvReaderFormat API to accept a factory/generator for the mapper/schema respectively, to support mappers that are using non-serializable extensions (like the `JavaTimeModule`).


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] zentol merged pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
zentol merged PR #20332:
URL: https://github.com/apache/flink/pull/20332


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] zentol commented on pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
zentol commented on PR #20332:
URL: https://github.com/apache/flink/pull/20332#issuecomment-1193857024

   @flinkbot run azure


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] zentol commented on pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
zentol commented on PR #20332:
URL: https://github.com/apache/flink/pull/20332#issuecomment-1192596936

   @flinkbot run azure


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] zentol commented on pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
zentol commented on PR #20332:
URL: https://github.com/apache/flink/pull/20332#issuecomment-1194266048

   We could add a dedicated test for the serializability of the format, but if this were violated it would already fail on the current master, and even more so after FLINK-28621 is merged (always registering date/time & optional extensions).


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] flinkbot commented on pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
flinkbot commented on PR #20332:
URL: https://github.com/apache/flink/pull/20332#issuecomment-1191376182

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "69db908ec2b2143dfa565d4f2b8641f8d6040a58",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "69db908ec2b2143dfa565d4f2b8641f8d6040a58",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 69db908ec2b2143dfa565d4f2b8641f8d6040a58 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run azure` re-run the last Azure build
   </details>


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [flink] zentol commented on pull request #20332: [FLINK-28624][csv] Accept mapper/schema factories

Posted by GitBox <gi...@apache.org>.
zentol commented on PR #20332:
URL: https://github.com/apache/flink/pull/20332#issuecomment-1195206311

   @flinkbot run azure


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org