You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2020/08/06 06:19:58 UTC

[GitHub] [shardingsphere] VincentChi opened a new issue #6661: Statement setFetchDirection not supported

VincentChi opened a new issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661


   ## Feature Request
   setFetchDirection is not supported and would throw exception
   org.apache.shardingsphere.shardingjdbc.jdbc.unsupported.AbstractUnsupportedOperationStatement#setFetchDirection
   
   ### Is your feature request related to a problem?
   We are using apache mahout which uses this method in db initialization, but would throw exception.
   ![image](https://user-images.githubusercontent.com/1518651/89497565-2e2bcf00-d7ef-11ea-8d9d-b0b164982893.png)
   
   ### Describe the feature you would like.
   It's better to support this feature, or just make it not final which could be overridden in our codes.
   **
   ![image](https://user-images.githubusercontent.com/1518651/89497684-6b905c80-d7ef-11ea-8fd4-3f4c39e41a65.png)
   **


----------------------------------------------------------------
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] [shardingsphere] wenweibin commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
wenweibin commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-669886254


   > Thanks for your debug. @VincentChi
   > We welcome you or other guys to try to fix this issue, for it seems not difficult.
   > You can remove `final` from `setFetchDirection` and implement it in `AbstractPreparedStatementAdapter` and `ShardingSpherePreparedStatement`.
   > 
   > If you need any help, please feel free to ping us!
   > 
   > Best,
   > Trista
   
   Can this be assgined to me?


----------------------------------------------------------------
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] [shardingsphere] tristaZero commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
tristaZero commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-669834409


   Thanks for your debug. @VincentChi 
   We welcome you or other guys to try to fix this issue, for it seems not difficult.
   You can remove `final` from `setFetchDirection` and implement it in `AbstractPreparedStatementAdapter` and `ShardingSpherePreparedStatement`.
   
   If you need any help, please feel free to ping us!
   
   Best,
   Trista
   


----------------------------------------------------------------
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] [shardingsphere] tristaZero commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
tristaZero commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-670390268


   My pleasure. @wenweibin 


----------------------------------------------------------------
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] [shardingsphere] kimmking commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
kimmking commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-696073805


   @wenweibin any improvement or close this issue for a long 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.

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

[GitHub] [shardingsphere] terrymanu commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
terrymanu commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-678899436


   @wenweibin Hi, any update?


----------------------------------------------------------------
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] [shardingsphere] kimmking commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
kimmking commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-696073805


   @wenweibin any improvement or close this issue for a long 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.

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

[GitHub] [shardingsphere] yu199195 commented on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
yu199195 commented on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-669731125


   we not support setFetchDirection，maybe cannot integrateion apache mahout， 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] [shardingsphere] wenweibin edited a comment on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
wenweibin edited a comment on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-669886254


   > Thanks for your debug. @VincentChi
   > We welcome you or other guys to try to fix this issue, for it seems not difficult.
   > You can remove `final` from `setFetchDirection` and implement it in `AbstractPreparedStatementAdapter` and `ShardingSpherePreparedStatement`.
   > 
   > If you need any help, please feel free to ping us!
   > 
   > Best,
   > Trista
   
   Can this be assgined to me? I want to try fixing it：）


----------------------------------------------------------------
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] [shardingsphere] tristaZero edited a comment on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
tristaZero edited a comment on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-670390268


   My pleasure. @wenweibin 
   You can request my review at your incoming PR. Good luck. :)


----------------------------------------------------------------
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] [shardingsphere] wenweibin edited a comment on issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
wenweibin edited a comment on issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661#issuecomment-669886254


   > Thanks for your debug. @VincentChi
   > We welcome you or other guys to try to fix this issue, for it seems not difficult.
   > You can remove `final` from `setFetchDirection` and implement it in `AbstractPreparedStatementAdapter` and `ShardingSpherePreparedStatement`.
   > 
   > If you need any help, please feel free to ping us!
   > 
   > Best,
   > Trista
   
   Can this be assgined to me? I want to try fixing it：） @tristaZero 


----------------------------------------------------------------
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] [shardingsphere] terrymanu closed issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
terrymanu closed issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661


   


----------------------------------------------------------------
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] [shardingsphere] terrymanu closed issue #6661: Statement setFetchDirection not supported

Posted by GitBox <gi...@apache.org>.
terrymanu closed issue #6661:
URL: https://github.com/apache/shardingsphere/issues/6661


   


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