[GitHub] [drill] cgivre commented on pull request #2599: DRILL-4232 Support for EXCEPT and INTERSECT set operator

2022-08-27 Thread GitBox
cgivre commented on PR #2599: URL: https://github.com/apache/drill/pull/2599#issuecomment-1229353429 @Leon-WTF Is this ready for review? -- 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 spec

[GitHub] [drill] cgivre commented on pull request #2631: [MINOR UPDATE]: Disable lgtm build

2022-08-27 Thread GitBox
cgivre commented on PR #2631: URL: https://github.com/apache/drill/pull/2631#issuecomment-1229353510 @pjfanning The INFRA ticket was closed and LGTM is now no longer active on the repo. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [drill] Leon-WTF commented on pull request #2599: DRILL-4232 Support for EXCEPT and INTERSECT set operator

2022-08-27 Thread GitBox
Leon-WTF commented on PR #2599: URL: https://github.com/apache/drill/pull/2599#issuecomment-1229365001 > @Leon-WTF Is this ready for review? @cgivre Not yet, I'm handling the EXCEPT case, it needs to remove the duplicate records for probe side, I'm trying to add an Agg phase after set