[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14737860#comment-14737860
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

Github user ebegoli closed the pull request at:

https://github.com/apache/drill/pull/142


> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Kristine Hahn
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14737859#comment-14737859
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

Github user ebegoli commented on the pull request:

https://github.com/apache/drill/pull/142#issuecomment-139078307
  
Kristen fixed it. I am closing pull request.


> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Kristine Hahn
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14737573#comment-14737573
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

Github user kristinehahn commented on the pull request:

https://github.com/apache/drill/pull/142#issuecomment-139043772
  
Sorry I can't merge or close this pull request, ^ ("Only those with write 
access . . ."), but I fixed the problem. Please close this pull request. Thanks 
for flagging the mistake.


> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Bridget Bevens
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14728346#comment-14728346
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

Github user ebegoli commented on the pull request:

https://github.com/apache/drill/pull/142#issuecomment-137291949
  
I have pull request in place so just merge it :-)

On Wednesday, September 2, 2015, Kristine Hahn 
wrote:

> Thanks, I also fixed this in my repo 13 days ago
> 
https://github.com/kristinehahn/drill/tree/gh-pages/_docs/sql-reference/sql-commands,
> but haven't been able to get it committed.
>
> —
> Reply to this email directly or view it on GitHub
> .
>



> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Bridget Bevens
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14728343#comment-14728343
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

Github user kristinehahn commented on the pull request:

https://github.com/apache/drill/pull/142#issuecomment-137291323
  
Thanks, I also fixed this in my repo 13 days ago 
https://github.com/kristinehahn/drill/tree/gh-pages/_docs/sql-reference/sql-commands,
 but haven't been able to get it committed. 


> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Bridget Bevens
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DRILL-3736) Documentation for partition is misleading/wrong syntax

2015-09-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/DRILL-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14728286#comment-14728286
 ] 

ASF GitHub Bot commented on DRILL-3736:
---

GitHub user ebegoli opened a pull request:

https://github.com/apache/drill/pull/142

Issue DRILL-3736 - fixed syntax of partition by.

It was incorrectly listed as PARTITION_BY. Changed to PARTITION BY.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ebegoli/drill gh-pages

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/drill/pull/142.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #142


commit cc010fd64e3911e39b91f2e7f42f0d6aaf2e336b
Author: Edmon Begoli 
Date:   2015-09-03T00:25:26Z

Issue DRILL-3736 - fixed syntax of partition by. 

It was incorrectly listed as PARTITION_BY. Changed to PARTITION BY.




> Documentation for partition is misleading/wrong syntax
> --
>
> Key: DRILL-3736
> URL: https://issues.apache.org/jira/browse/DRILL-3736
> Project: Apache Drill
>  Issue Type: Bug
>  Components: Documentation
>Affects Versions: 1.1.0
> Environment: Any
>Reporter: Edmon Begoli
>Assignee: Bridget Bevens
>Priority: Minor
>  Labels: documentation
> Fix For: 1.2.0
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> Examples on the web site use appropriate syntax (PARTITION BY)
> but the syntax definition uses PARTITION_BY.
> https://drill.apache.org/docs/partition-by-clause/
> It should all be PARTITION BY.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)