You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2020/09/21 09:46:33 UTC

[GitHub] [drill] gdubya opened a new pull request #2104: Fix syntax error in "drill-config.sh"

gdubya opened a new pull request #2104:
URL: https://github.com/apache/drill/pull/2104


   The drill-config.sh has a syntax error on line 144 with the usage of "let". The command following let should be enclosed in quotes, otherwise the following syntax error occurs:
   /home/someone/proj/apache-drill-1.17.0/bin/drill-config.sh: line 144: let: lineCount=: syntax error: operand expected (error token is "=")
   
   # [DRILL-XXXX](https://issues.apache.org/jira/browse/DRILL-XXXX): PR Title
   
   (Please replace `PR Title` with actual PR Title)
   
   ## Description
   
   (Please describe the change. If more than one ticket is fixed, include a reference to those tickets.)
   
   ## Documentation
   (Please describe user-visible changes similar to what should appear in the Drill documentation.)
   
   ## Testing
   (Please describe how this PR has been tested.)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] gdubya commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
gdubya commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-696051895


   I'm not sure why this caused the build to fail.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] gdubya commented on pull request #2104: DRILL-7793: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
gdubya commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-702346610


   There was no JIRA issue. Created [DRILL-7793](https://issues.apache.org/jira/browse/DRILL-7793) 👍 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-698889555


   LGTM +1


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre merged pull request #2104: DRILL-7793: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre merged pull request #2104:
URL: https://github.com/apache/drill/pull/2104


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-698889269






----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] gdubya commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
gdubya commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-696051895


   I'm not sure why this caused the build to fail.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre commented on pull request #2104: DRILL-7793: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-702356066


   Thanks @gdubya!  


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-701554125


   @gdubya 
   Is there a JIRA for this?  If not, could you please create a JIRA issue at issues.apache.org and then put a link in the title of the PR?  Once that's done, I'll commit. 
   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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre commented on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-698889269


   I think the CI is just timing out.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [drill] cgivre removed a comment on pull request #2104: Fix syntax error in "drill-config.sh"

Posted by GitBox <gi...@apache.org>.
cgivre removed a comment on pull request #2104:
URL: https://github.com/apache/drill/pull/2104#issuecomment-698889555


   LGTM +1


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org