You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "xiangyuf (via GitHub)" <gi...@apache.org> on 2024/07/23 09:41:28 UTC
[PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
xiangyuf opened a new pull request, #37:
URL: https://github.com/apache/paimon-presto/pull/37
To close the issue: #36
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "xiangyuf (via GitHub)" <gi...@apache.org>.
xiangyuf commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2292656680
@JingsongLi @lanvendar Hi, in my humble opinion, we should revert the 0.287 upgrade first and introduce this later at least until it compiles.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #37:
URL: https://github.com/apache/paimon-presto/pull/37
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "lanvendar (via GitHub)" <gi...@apache.org>.
lanvendar commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2295771272
> @JingsongLi @lanvendar Hi, in my humble opinion, we should revert the 0.287 upgrade first and introduce this later at least until it compiles.
The testing bug has been fixed. Please refer to my fork personal repository for details. I will apply for a merge later
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "xiangyuf (via GitHub)" <gi...@apache.org>.
xiangyuf commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2296362963
> > @JingsongLi @lanvendar Hi, in my humble opinion, we should revert the 0.287 upgrade first and introduce this later at least until it compiles.
>
> @xiangyuf The testing bug has been fixed. Please refer to my fork personal repository for details. I will apply for a merge later 
@lanvendar Looking forward for your MR.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "xiangyuf (via GitHub)" <gi...@apache.org>.
xiangyuf commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2244741959
@JingsongLi @lanvendar pls review this when you have time.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "xiangyuf (via GitHub)" <gi...@apache.org>.
xiangyuf commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2292653936
> pls compile without test code,like this mvn clean install -DskipTests -am -pl paimon-presto-0.287 -Dpresto.version=0.287 -Dhadoop.apache2.version=2.7.4-12 -Dhive.apache.version=3.0.0-10
>
> > @JingsongLi @lanvendar pls review this when you have time.
Still error.
After revert 0.287, the compilation could success.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [Bugfix] Revert upgrade presto-0.287 [paimon-presto]
Posted by "lanvendar (via GitHub)" <gi...@apache.org>.
lanvendar commented on PR #37:
URL: https://github.com/apache/paimon-presto/pull/37#issuecomment-2257746479
pls compile without test code,like this
mvn clean install -DskipTests -Dmaven.test.skip=true -am -pl paimon-presto-0.287 -Dpresto.version=0.287 -Dhadoop.apache2.version=2.7.4-12 -Dhive.apache.version=3.0.0-10
> @JingsongLi @lanvendar pls review this when you have time.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org