[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2022-06-03 Thread Kenneth Knowles (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17547788#comment-17547788 ] Kenneth Knowles commented on BEAM-7049: --- This issue has been migrated to https://git

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2020-08-25 Thread Beam JIRA Bot (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17184452#comment-17184452 ] Beam JIRA Bot commented on BEAM-7049: - This issue was marked "stale-P2" and has not re

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2020-08-10 Thread Beam JIRA Bot (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17174741#comment-17174741 ] Beam JIRA Bot commented on BEAM-7049: - This issue is P2 but has been unassigned withou

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2020-06-10 Thread Beam JIRA Bot (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17131385#comment-17131385 ] Beam JIRA Bot commented on BEAM-7049: - This issue was marked "stale-assigned" and has

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2020-06-01 Thread Kenneth Knowles (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17122065#comment-17122065 ] Kenneth Knowles commented on BEAM-7049: --- This issue is assigned but has not received

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-10-07 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16946252#comment-16946252 ] sridhar Reddy commented on BEAM-7049: - Thank you for the update.  re: Infinite Loop.

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-10-03 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16944063#comment-16944063 ] Rui Wang commented on BEAM-7049: Ah ok. I found a tricky problem: given the merged cost ba

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-10-02 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16943253#comment-16943253 ] sridhar Reddy commented on BEAM-7049: - Sounds good. Take your time. I appreciate you h

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-10-02 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16943072#comment-16943072 ] Rui Wang commented on BEAM-7049: Sorry [~sridharG] I probably have missed your message and

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-30 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16941410#comment-16941410 ] sridhar Reddy commented on BEAM-7049: - [~amaliujia] I just want to make sure that we a

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-19 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933828#comment-16933828 ] sridhar Reddy commented on BEAM-7049: - I reviewed the PR and related JIRA. I see what

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-18 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933052#comment-16933052 ] Rui Wang commented on BEAM-7049: Recently there was a effort to separate multiple join imp

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-18 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933048#comment-16933048 ] sridhar Reddy commented on BEAM-7049: - I agree with your assessment.  > Merge multipl

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-18 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933047#comment-16933047 ] Rui Wang commented on BEAM-7049: Hmm this JIRA becomes more complicated than I imagined.

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-18 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933042#comment-16933042 ] sridhar Reddy commented on BEAM-7049: - [~amaliujia] I tried the tests you suggested an

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-18 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933041#comment-16933041 ] sridhar Reddy commented on BEAM-7049: - I did the tests with the following steps # re-

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-17 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16932032#comment-16932032 ] sridhar Reddy commented on BEAM-7049: - I noticed there is a change in behavior between

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929541#comment-16929541 ] sridhar Reddy commented on BEAM-7049: - I believe all the code that is required for thi

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929500#comment-16929500 ] sridhar Reddy commented on BEAM-7049: - With the change in location of UnionMergeRule.I

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929490#comment-16929490 ] sridhar Reddy commented on BEAM-7049: - Reviewed your comment on PR. I will change the

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929477#comment-16929477 ] Rui Wang commented on BEAM-7049: Interesting. I left a comment on your PR. > Merge multip

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929478#comment-16929478 ] sridhar Reddy commented on BEAM-7049: - My WIP pull request (#9358 )already contains co

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929474#comment-16929474 ] sridhar Reddy commented on BEAM-7049: - My inputs here are 2 https://github.com/apache

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-13 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929466#comment-16929466 ] sridhar Reddy commented on BEAM-7049: - For the query "SELECT 1 UNION ALL SELECT 2 UNIO

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-12 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16929007#comment-16929007 ] Rui Wang commented on BEAM-7049: I can then observe that there are 5 inputs at here: http

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-12 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16928996#comment-16928996 ] Rui Wang commented on BEAM-7049: I used query "SELECT 1 UNION ALL SELECT 2 UNION ALL SELE

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-12 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16928956#comment-16928956 ] sridhar Reddy commented on BEAM-7049: - Sounds good! > Merge multiple input to one Bea

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-12 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16928954#comment-16928954 ] Rui Wang commented on BEAM-7049: [~sridharG] I will do some testing on my side to check U

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-03 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16921535#comment-16921535 ] sridhar Reddy commented on BEAM-7049: - Hope you had a nice vacation! I will be on vaca

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-09-03 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16921523#comment-16921523 ] Rui Wang commented on BEAM-7049: Hello, I was on a vocation and I am catching up pending e

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-29 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16919139#comment-16919139 ] sridhar Reddy commented on BEAM-7049: - After several attempts to generalize union with

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-22 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16913442#comment-16913442 ] sridhar Reddy commented on BEAM-7049: - Thanks for the quick update. I agree that it is

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-21 Thread Rui Wang (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16912980#comment-16912980 ] Rui Wang commented on BEAM-7049: Although usually most of the use case we might only need

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-21 Thread sridhar Reddy (Jira)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16912882#comment-16912882 ] sridhar Reddy commented on BEAM-7049: - Just including UnionMergeRule.INSTANCE in BeamR

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-15 Thread sridhar Reddy (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16908680#comment-16908680 ] sridhar Reddy commented on BEAM-7049: - Here is the pull request. As I mentioned in the

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-14 Thread sridhar Reddy (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16907689#comment-16907689 ] sridhar Reddy commented on BEAM-7049: - Good idea! I will create one > Merge multiple

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-13 Thread Rui Wang (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16906758#comment-16906758 ] Rui Wang commented on BEAM-7049: [~sridharG] I would also encourage you keep a WIP PR in

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-13 Thread sridhar Reddy (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16906734#comment-16906734 ] sridhar Reddy commented on BEAM-7049: - Made a couple of simple cases work  1)select 1

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-12 Thread Rui Wang (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16905707#comment-16905707 ] Rui Wang commented on BEAM-7049: Sounds good! BTW you might need to enable [1] in [2] [1

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-12 Thread sridhar Reddy (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16905695#comment-16905695 ] sridhar Reddy commented on BEAM-7049: - Made great progress in understanding the code b

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-08 Thread sridhar Reddy (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16903422#comment-16903422 ] sridhar Reddy commented on BEAM-7049: - [~amaliujia] Thanks for the update. I think I u

[jira] [Commented] (BEAM-7049) Merge multiple input to one BeamUnionRel

2019-08-07 Thread Rui Wang (JIRA)
[ https://issues.apache.org/jira/browse/BEAM-7049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16902560#comment-16902560 ] Rui Wang commented on BEAM-7049: And actually I think this JIRA's title was misleading, mu