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

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

                Author: ASF GitHub Bot
            Created on: 19/Apr/22 13:22
            Start Date: 19/Apr/22 13:22
    Worklog Time Spent: 10m 
      Work Description: riteshghorse commented on code in PR #17370:
URL: https://github.com/apache/beam/pull/17370#discussion_r853068133


##########
sdks/go/pkg/beam/core/graph/coder/panes.go:
##########
@@ -60,11 +64,11 @@ func EncodePane(v typex.PaneInfo, w io.Writer) error {
 func NewPane(b byte) typex.PaneInfo {
        pn := typex.NoFiringPane()
 
-       if b&0x01 == 1 {
-               pn.IsFirst = true
+       if !(b&0x02 == 2) {
+               pn.IsFirst = false

Review Comment:
   This should be set as true?





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

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

> Add unit testing to the pane coder
> ----------------------------------
>
>                 Key: BEAM-14306
>                 URL: https://issues.apache.org/jira/browse/BEAM-14306
>             Project: Beam
>          Issue Type: Sub-task
>          Components: sdk-go
>            Reporter: Jack McCluskey
>            Assignee: Jack McCluskey
>            Priority: P2
>          Time Spent: 2h
>  Remaining Estimate: 0h
>
> Add unit testing to the pane code in graph/coder. Coder implementation 
> follows the [runner API 
> spec|https://github.com/apache/beam/blob/master/model/pipeline/src/main/proto/org/apache/beam/model/pipeline/v1/beam_runner_api.proto#L922].
>  



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to