You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by "strongduanmu (via GitHub)" <gi...@apache.org> on 2023/03/01 08:35:51 UTC

[GitHub] [shardingsphere] strongduanmu opened a new issue, #24403: Remove useless SKIP_ENCRYPT_REWRITE SQL HINT

strongduanmu opened a new issue, #24403:
URL: https://github.com/apache/shardingsphere/issues/24403

   ## Feature Request
   
   ### Is your feature request related to a problem?
   
   #15965
   
   ### Describe the feature you would like.
   
   Currently, ShardingSphere provides `/* SHARDINGSPHERE_HINT: SKIP_ENCRYPT_REWRITE=false */` to skip the encrypt rewriting process, thus supporting clearing of plaintext columns. But when `/* SHARDINGSPHERE_HINT: DATA_SOURCE_NAME=ds_0 */` appears, `/* SHARDINGSPHERE_HINT: SKIP_ENCRYPT_REWRITE=false */` is somewhat redundant, we can use `/* SHARDINGSPHERE_HINT: DATA_SOURCE_NAME=ds_0 */` to achieve the same function. So it's time to remove 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.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org.apache.org

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

[GitHub] [shardingsphere] strongduanmu commented on issue #24403: Remove useless SKIP_ENCRYPT_REWRITE SQL HINT

Posted by "strongduanmu (via GitHub)" <gi...@apache.org>.
strongduanmu commented on issue #24403:
URL: https://github.com/apache/shardingsphere/issues/24403#issuecomment-1449654560

   I did some research, currently `/* SHARDINGSPHERE_HINT: DATA_SOURCE_NAME=ds_0 */` will only be processed for data source routing, and SQL rewriting cannot intervene, so a better way is to use it with `/* SHARDINGSPHERE_HINT: DATA_SOURCE_NAME=ds_0 * /` and `/* SHARDINGSPHERE_HINT: SKIP_ENCRYPT_REWRITE=false */` , so as to achieve more flexible control.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

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

[GitHub] [shardingsphere] zhaojinchao95 closed issue #24403: Adapt `SKIP_ENCRYPT_REWRITE` to `SKIP_SQL_REWRITE` for more scenarios

Posted by "zhaojinchao95 (via GitHub)" <gi...@apache.org>.
zhaojinchao95 closed issue #24403: Adapt `SKIP_ENCRYPT_REWRITE` to `SKIP_SQL_REWRITE` for more scenarios
URL: https://github.com/apache/shardingsphere/issues/24403


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

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