You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by GitBox <gi...@apache.org> on 2021/07/24 11:14:43 UTC
[GitHub] [hive] ujc714 opened a new pull request #2255: HIVE-25095: Beeline/hive -e command can't deal with query with traili…
ujc714 opened a new pull request #2255:
URL: https://github.com/apache/hive/pull/2255
…### What changes were proposed in this pull request?
<!--
Please clarify what changes you are proposing. The purpose of this section is to outline the changes and how this PR fixes the issue.
If possible, please consider writing useful notes for better and faster reviews in your PR. See the examples below.
1. If you refactor some codes with changing classes, showing the class hierarchy will help reviewers.
2. If you fix some SQL features, you can provide some references of other DBMSes.
3. If there is design documentation, please add the link.
4. If there is a discussion in the mailing list, please add the link.
-->
Upgrade commons-cli to 1.4.
### Why are the changes needed?
There is a bug in commons-cli-1.2 which trips trailing quote improperly.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
mvn test -Dtest=org.apache.hive.beeline.cli.TestHiveCli
mvn test -Dtest=org.apache.hive.beeline.TestBeelineArgParsing
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ujc714 edited a comment on pull request #2255: HIVE-25095: Beeline/hive -e command can't deal with query with traili…
Posted by GitBox <gi...@apache.org>.
ujc714 edited a comment on pull request #2255:
URL: https://github.com/apache/hive/pull/2255#issuecomment-886038213
The branch was lost in my fork for some reason. I restored it and had to open a new PR so close this one. Could you please review PR-2526 after it passes tests? Thanks!
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ujc714 commented on pull request #2255: HIVE-25095: Beeline/hive -e command can't deal with query with traili…
Posted by GitBox <gi...@apache.org>.
ujc714 commented on pull request #2255:
URL: https://github.com/apache/hive/pull/2255#issuecomment-886038213
The branch was lost in my fork for some reason. I restored it and have to open a new PR so close this one. Could you please review PR-2526 after it passes tests? Thanks!
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ujc714 closed pull request #2255: HIVE-25095: Beeline/hive -e command can't deal with query with traili…
Posted by GitBox <gi...@apache.org>.
ujc714 closed pull request #2255:
URL: https://github.com/apache/hive/pull/2255
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org
[GitHub] [hive] ujc714 commented on pull request #2255: HIVE-25095: Beeline/hive -e command can't deal with query with traili…
Posted by GitBox <gi...@apache.org>.
ujc714 commented on pull request #2255:
URL: https://github.com/apache/hive/pull/2255#issuecomment-886038014
> LGTM +1, please squash your commits and rebase on top of the current master to get a clean test run
--
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: gitbox-unsubscribe@hive.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org