[GitHub] [spark] aokolnychyi commented on pull request #33096: [SPARK-35899][SQL] Utility to convert connector expressions to Catalyst

2021-06-29 Thread GitBox
aokolnychyi commented on pull request #33096: URL: https://github.com/apache/spark/pull/33096#issuecomment-869982159 @HyukjinKwon, thanks for taking a look! A few clarifying questions and I'll submit a follow-up PR. -- This is an automated message from the Apache Git Service. To

[GitHub] [spark] aokolnychyi commented on pull request #33096: [SPARK-35899][SQL] Utility to convert connector expressions to Catalyst

2021-06-28 Thread GitBox
aokolnychyi commented on pull request #33096: URL: https://github.com/apache/spark/pull/33096#issuecomment-869982159 @HyukjinKwon, thanks for taking a look! A few clarifying questions and I'll submit a follow-up PR. -- This is an automated message from the Apache Git Service. To

[GitHub] [spark] aokolnychyi commented on pull request #33096: [SPARK-35899][SQL] Utility to convert connector expressions to Catalyst

2021-06-25 Thread GitBox
aokolnychyi commented on pull request #33096: URL: https://github.com/apache/spark/pull/33096#issuecomment-868900677 Thank you, @dongjoon-hyun! -- 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

[GitHub] [spark] aokolnychyi commented on pull request #33096: [SPARK-35899][SQL] Utility to convert connector expressions to Catalyst

2021-06-25 Thread GitBox
aokolnychyi commented on pull request #33096: URL: https://github.com/apache/spark/pull/33096#issuecomment-868897551 Good call, @dongjoon-hyun. Added to the PR description. Could you check, please? -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [spark] aokolnychyi commented on pull request #33096: [SPARK-35899][SQL] Utility to convert connector expressions to Catalyst

2021-06-25 Thread GitBox
aokolnychyi commented on pull request #33096: URL: https://github.com/apache/spark/pull/33096#issuecomment-868882776 This PR contains a utility class I need for dynamic filtering. cc @sunchao @huaxingao @viirya @dongjoon-hyun @cloud-fan @HyukjinKwon @rdblue @holdenk -- This is an