trino icon indicating copy to clipboard operation
trino copied to clipboard

fix compiler failure with regexp_replace function when working with Hive tables containing row(map<varchar, varchar>) data type

Open needlssj opened this issue 1 year ago • 13 comments

Additional context and related issues

fix #19689

needlssj avatar Nov 15 '23 08:11 needlssj

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

cla-bot[bot] avatar Nov 15 '23 08:11 cla-bot[bot]

Cla pdf on appendix

------------------ Original ------------------ From: cla-bot[bot] @.> Date: Wed,Nov 15,2023 4:06 PM To: trinodb/trino @.> Cc: needlssj @.>, Author @.> Subject: Re: [trinodb/trino] fix compiler failure with regexp_replace functionwhen working with Hive tables containing row(map<varchar, varchar>) data type(PR #19749)

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to @.*** Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

needlssj avatar Nov 15 '23 08:11 needlssj

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

cla-bot[bot] avatar Nov 15 '23 08:11 cla-bot[bot]

This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua

github-actions[bot] avatar Jan 10 '24 17:01 github-actions[bot]

:wave: @needlssj - this PR has become inactive. If you're still interested in working on it, please let us know, and we can try to get reviewers to help with that.

We're working on closing out old and inactive PRs, so if you're too busy or this has too many merge conflicts to be worth picking back up, we'll be making another pass to close it out in a few weeks.

mosabua avatar Jan 11 '24 19:01 mosabua

This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua

github-actions[bot] avatar Feb 02 '24 17:02 github-actions[bot]

@findepi @findinpath @alexjo2144 @electrum is there merit in this PR and fix or should we close?

mosabua avatar Feb 02 '24 20:02 mosabua

This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua

github-actions[bot] avatar Feb 28 '24 17:02 github-actions[bot]

Apologies for the delay in reviewing this. Can you share a reproduction?

electrum avatar Feb 28 '24 23:02 electrum

This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua

github-actions[bot] avatar Mar 26 '24 17:03 github-actions[bot]

Closing this pull request, as it has been stale for six weeks. Feel free to re-open at any time.

github-actions[bot] avatar Apr 16 '24 17:04 github-actions[bot]

Reopened and added stale-ignore .. fyi @findepi and @electrum

mosabua avatar Apr 17 '24 16:04 mosabua

This might have been solved by https://github.com/trinodb/trino/pull/20226.

takezoe avatar Jun 27 '24 23:06 takezoe

@takezoe indeed, thanks.

hashhar avatar Jul 01 '24 06:07 hashhar