roger-mike commented on PR #16911: URL: https://github.com/apache/beam/pull/16911#issuecomment-1116463231
> Unfortunately your link is broken. Is this the index? Looking at the generated code, I think hardcoding an index of `0` is reasonable as analytic functions only have one signature. https://github.com/apache/beam/pull/16911/files#diff-b497391b74a7130be5217a2d4ede920fe66059444bc248a6710ad9b6361ac2b3R105 Sorry about that, this is the index I was talking about: https://github.com/apache/beam/blob/f83ced3b8a336bd583fea5d0c352b419bfb0899b/sdks/java/extensions/sql/zetasql/src/main/java/org/apache/beam/sdk/extensions/sql/zetasql/translation/ExpressionConverter.java#L213 -- 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]
