[ 
https://issues.apache.org/jira/browse/BEAM-7428?focusedWorklogId=251141&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-251141
 ]

ASF GitHub Bot logged work on BEAM-7428:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 30/May/19 17:26
            Start Date: 30/May/19 17:26
    Worklog Time Spent: 10m 
      Work Description: kennknowles commented on issue #8691: [BEAM-7428] 
Output the timestamp on elements in ReadAllViaFileBasedSource
URL: https://github.com/apache/beam/pull/8691#issuecomment-497412367
 
 
   The stacktrace you posted was not the root cause. It is helpful to know the 
stacktrace of the IllegalArgumentException. The assertion is likely preventing 
output of data that would just be droppable anyhow. The error message does not 
seem quite right, but I bet there is danger 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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 251141)
    Time Spent: 2h  (was: 1h 50m)

> ReadAllViaFileBasedSource does not output the timestamps of the read elements
> -----------------------------------------------------------------------------
>
>                 Key: BEAM-7428
>                 URL: https://issues.apache.org/jira/browse/BEAM-7428
>             Project: Beam
>          Issue Type: Bug
>          Components: sdk-java-core
>            Reporter: Ismaël Mejía
>            Assignee: Ismaël Mejía
>            Priority: Minor
>          Time Spent: 2h
>  Remaining Estimate: 0h
>
> This differs from the implementation of JavaReadViaImpulse that tackles a 
> similar problem but does output the timestamps correctly.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to