You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@tajo.apache.org by hys9958 <gi...@git.apache.org> on 2014/12/02 08:50:39 UTC

[GitHub] tajo pull request: TAJO-1220: Implement JDBC function for 'createS...

GitHub user hys9958 opened a pull request:

    https://github.com/apache/tajo/pull/276

    TAJO-1220: Implement JDBC function for 'createStatement' and 'setEscapeP...

    Hi guys!
    - Statement createStatement(int resultSetType, int resultSetConcurrency)
       - Support 'CONCUR_READ_ONLY' only. 
       - Not support 'TYPE_SCROLL_SENSITIVE'.
    
    - void setEscapeProcessing(boolean enable)
       - Tajo is not necessary the function. So do nothing.

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

    $ git pull https://github.com/hys9958/tajo tajo-1220

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

    https://github.com/apache/tajo/pull/276.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 #276
    
----
commit ffd5aa0dd188d35214f9ea1edc792a4e77184b32
Author: hys9958 <ha...@gmail.com>
Date:   2014-12-02T07:04:43Z

    TAJO-1220: Implement JDBC function for 'createStatement' and 'setEscapeProcessing'.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1220: Implement JDBC function for 'createS...

Posted by hys9958 <gi...@git.apache.org>.

Github user hys9958 commented on the pull request:

    https://github.com/apache/tajo/pull/276#issuecomment-65364879
  
    Thanks


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1220: Implement JDBC function for 'createS...

Posted by hyunsik <gi...@git.apache.org>.

Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/276#issuecomment-65363952
  
    +1
    In overall, the fix looks straightforward. I've slightly changed the comments and some codes. I'll commit it shortly.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tajo pull request: TAJO-1220: Implement JDBC function for 'createS...

Posted by hys9958 <gi...@git.apache.org>.

Github user hys9958 closed the pull request at:

    https://github.com/apache/tajo/pull/276


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---