KomachiSion merged pull request #4198: Remove empty test cases for
VisitorParameterizedParsingTest
URL: https://github.com/apache/incubator-shardingsphere/pull/4198
This is an automated message from the Apache Git Service.
terrymanu opened a new pull request #4198: Remove empty test cases for
VisitorParameterizedParsingTest
URL: https://github.com/apache/incubator-shardingsphere/pull/4198
For #3947.
This is an automated message from the Apache
xiyelife commented on issue #4188: sharding +encrypt rewrite for SQL `select
with `OR` and `AND`` cause StringIndexOutOfBoundsException.
URL:
https://github.com/apache/incubator-shardingsphere/issues/4188#issuecomment-583270289
where's predicate contains `OR` operator that makes encrypt
xiyelife edited a comment on issue #4188: sharding +encrypt rewrite for SQL
`select with `OR` and `AND`` cause StringIndexOutOfBoundsException.
URL:
https://github.com/apache/incubator-shardingsphere/issues/4188#issuecomment-583270289
where predicate contains `OR` operator that makes
tuohai666 merged pull request #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
kimmking commented on a change in pull request #4197: Travis config
URL:
https://github.com/apache/incubator-shardingsphere/pull/4197#discussion_r376240759
##
File path: .travis.yml
##
@@ -1,15 +1,21 @@
-sudo: required
-cache:
- directories:
-- "$HOME/.m2"
-
tuohai666 commented on issue #4197: Travis config
URL:
https://github.com/apache/incubator-shardingsphere/pull/4197#issuecomment-583259296
Please add labels and milestone.
I think this pr is also applicable for 4.0.1 branch.
tuohai666 commented on a change in pull request #4197: Travis config
URL:
https://github.com/apache/incubator-shardingsphere/pull/4197#discussion_r376237092
##
File path: .travis.yml
##
@@ -1,15 +1,21 @@
-sudo: required
-cache:
- directories:
-- "$HOME/.m2"
-
tristaZero merged pull request #4195: Remove SQLVisitor
URL: https://github.com/apache/incubator-shardingsphere/pull/4195
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
tristaZero closed issue #3914: Optimization for Sharing Parser with Antlr
Visitor
URL: https://github.com/apache/incubator-shardingsphere/issues/3914
This is an automated message from the Apache Git Service.
To respond to
coveralls commented on issue #4195: Remove SQLVisitor
URL:
https://github.com/apache/incubator-shardingsphere/pull/4195#issuecomment-583236419
## Pull Request Test Coverage Report for [Build
9355](https://coveralls.io/builds/28582499)
* **0** of **1** **(0.0%)** changed or added
terrymanu commented on issue #4196: old DCL set default role test unit dose not
pass
URL:
https://github.com/apache/incubator-shardingsphere/issues/4196#issuecomment-583234268
This is not bug, it is a normal assert.
The failure reason just like the error message said:
```
The
terrymanu closed issue #4196: old DCL set default role test unit dose not pass
URL: https://github.com/apache/incubator-shardingsphere/issues/4196
This is an automated message from the Apache Git Service.
To respond to the
beijing-penguin opened a new issue #4196: set default role old test dose not
pass
URL: https://github.com/apache/incubator-shardingsphere/issues/4196
i add set-default-role.xml to
incubator-shardingsphere\shardingsphere-sql-parser\shardingsphere-sql-parser-test\src\test\resources\sql,but
terrymanu opened a new pull request #4195: Remove SQLVisitor
URL: https://github.com/apache/incubator-shardingsphere/pull/4195
Fixes #3914.
Use ParseTreeVisitor instead of SQLVisitor. Avoid create a new interface to
increase complexity.
xiyelife commented on issue #4037: sharding+encrypt when sql contains "\n"，sql
rewrite error for mybatis. （in）
URL:
https://github.com/apache/incubator-shardingsphere/issues/4037#issuecomment-583221283
@SteNicholas I add parse logic of the IN operator bracket's index for
encrypt
tristaZero commented on issue #3914: Optimization for Sharing Parser with Antlr
Visitor
URL:
https://github.com/apache/incubator-shardingsphere/issues/3914#issuecomment-583220969
Here are some remaining task from my position,
MySQL visitor progress:
- DAL **(In charge of
tristaZero commented on a change in pull request #4191: set role and set
default role visit parse
URL:
https://github.com/apache/incubator-shardingsphere/pull/4191#discussion_r376197916
##
File path:
tristaZero commented on a change in pull request #4190: Add mysql grant visitor
for DCLStatement
URL:
https://github.com/apache/incubator-shardingsphere/pull/4190#discussion_r376194678
##
File path:
tristaZero merged pull request #4194: Rename ListValue to CollectionValue
URL: https://github.com/apache/incubator-shardingsphere/pull/4194
This is an automated message from the Apache Git Service.
To respond to the message,
xiyelife commented on a change in pull request #4193: Fixes #4037. IN operator
contains space and `\n` `\t` `\r`
URL:
https://github.com/apache/incubator-shardingsphere/pull/4193#discussion_r376192010
##
File path:
dongzl commented on issue #4166: refactor orchestration config for apollo.
URL:
https://github.com/apache/incubator-shardingsphere/pull/4166#issuecomment-583190548
/run ci
This is an automated message from the Apache Git
dongzl removed a comment on issue #4166: refactor orchestration config for
apollo.
URL:
https://github.com/apache/incubator-shardingsphere/pull/4166#issuecomment-583190548
/run ci
This is an automated message from the
This is an automated email from the ASF dual-hosted git repository.
zhaoyanan pushed a commit to branch master
in repository
https://gitbox.apache.org/repos/asf/incubator-shardingsphere-benchmark.git
The following commit(s) were added to refs/heads/master by this push:
new 2548fa9 for
coveralls edited a comment on issue #4194: Rename ListValue to CollectionValue
URL:
https://github.com/apache/incubator-shardingsphere/pull/4194#issuecomment-583033596
## Pull Request Test Coverage Report for [Build
9350](https://coveralls.io/builds/28573084)
* **0** of **11**
coveralls commented on issue #4194: Rename ListValue to CollectionValue
URL:
https://github.com/apache/incubator-shardingsphere/pull/4194#issuecomment-583033596
## Pull Request Test Coverage Report for [Build
1627](https://coveralls.io/builds/28572284)
* **0** of **11**
terrymanu commented on a change in pull request #4193: Fixes #4037. IN operator
contains space and `\n` `\t` `\r`
URL:
https://github.com/apache/incubator-shardingsphere/pull/4193#discussion_r375971137
##
File path:
terrymanu opened a new pull request #4194: Rename ListValue to CollectionValue
URL: https://github.com/apache/incubator-shardingsphere/pull/4194
For #3914.
This is an automated message from the Apache Git Service.
To
xiyelife opened a new pull request #4193: Fixes #4037. IN operator contains
space and `\n` `\t` `\r`
URL: https://github.com/apache/incubator-shardingsphere/pull/4193
Fixes #4037.
Changes proposed in this pull request:
- parse the IN operator bracket's index for encrypt rewirte
coveralls commented on issue #4190: Add mysql grant visitor for DCLStatement
URL:
https://github.com/apache/incubator-shardingsphere/pull/4190#issuecomment-582924391
## Pull Request Test Coverage Report for [Build
9348](https://coveralls.io/builds/28567283)
* **0** of **19**
lklkxcxc opened a new issue #4192: insert float type vaule report "10002 -
2Unknown exception"
URL: https://github.com/apache/incubator-shardingsphere/issues/4192
Sharding-proxy version: 4.0.0-RC1
Problems encountered：
mysql import sql found insert and delete loat type vaule
beijing-penguin opened a new pull request #4191: set role and set default role
visit parse
URL: https://github.com/apache/incubator-shardingsphere/pull/4191
Changes proposed in this pull request:
- add set default role grammar visit and xml test
- fix set role sql grammar error
tianbin1001 opened a new pull request #4190: Add mysql grant visitor for
DCLStatement
URL: https://github.com/apache/incubator-shardingsphere/pull/4190
Add mysql grant visitor for DCLStatement
This is an automated message
coveralls edited a comment on issue #4186: modify createInSegment() for select
URL:
https://github.com/apache/incubator-shardingsphere/pull/4186#issuecomment-582874056
## Pull Request Test Coverage Report for [Build
9343](https://coveralls.io/builds/28565665)
* **0** of **24**
cielswift opened a new issue #3993: Start error with yml
URL: https://github.com/apache/incubator-shardingsphere/issues/3993
spring boot version 2.2
jdk 13
sharding 4.0.0-RC3
I searched most of the cases and configured them with properties
But I used yml, but something went
terrymanu commented on issue #3993: Start error with yml
URL:
https://github.com/apache/incubator-shardingsphere/issues/3993#issuecomment-582879925
Any update?
This is an automated message from the Apache Git Service.
To
terrymanu closed issue #3993: Start error with yml
URL: https://github.com/apache/incubator-shardingsphere/issues/3993
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
terrymanu commented on issue #3971: Why does the encrypted table only judge the
first one, maybe left join encrypt table, and then judge that there is no
encrypted table
URL:
https://github.com/apache/incubator-shardingsphere/issues/3971#issuecomment-582879471
Can you provide your SQL
terrymanu closed issue #3970: Got many warn message:
com.mysql.jdbc.jdbc2.optional.MysqlXAException: Undetermined error occurred in
the underlying Connection - check your data for consistency
URL: https://github.com/apache/incubator-shardingsphere/issues/3970
terrymanu closed issue #3927: Whether the database field desensitization logic
can only process the SQL of the specified encrypted table without disturbing
the normal operation of other SQL
URL: https://github.com/apache/incubator-shardingsphere/issues/3927
terrymanu closed issue #3919: config datasource dynamic by program or outside
order？
URL: https://github.com/apache/incubator-shardingsphere/issues/3919
This is an automated message from the Apache Git Service.
To respond
terrymanu commented on issue #3927: Whether the database field desensitization
logic can only process the SQL of the specified encrypted table without
disturbing the normal operation of other SQL
URL:
https://github.com/apache/incubator-shardingsphere/issues/3927#issuecomment-582878606
terrymanu closed issue #3886: Can we consider sharing the pool of sharded data
source connections？
URL: https://github.com/apache/incubator-shardingsphere/issues/3886
This is an automated message from the Apache Git
terrymanu commented on issue #4008: The insert statement reported an error
URL:
https://github.com/apache/incubator-shardingsphere/issues/4008#issuecomment-582877975
Closed for no response anymore.
This is an automated
terrymanu closed issue #4008: The insert statement reported an error
URL: https://github.com/apache/incubator-shardingsphere/issues/4008
This is an automated message from the Apache Git Service.
To respond to the message,
terrymanu commented on issue #4031: Duplicate entry '425919356283650049' for
key 'PRIMARY'
URL:
https://github.com/apache/incubator-shardingsphere/issues/4031#issuecomment-582877756
Closed for no response anymore.
This is
terrymanu closed issue #4053: default-data-source-name
URL: https://github.com/apache/incubator-shardingsphere/issues/4053
This is an automated message from the Apache Git Service.
To respond to the message, please log on
terrymanu closed issue #4031: Duplicate entry '425919356283650049' for key
'PRIMARY'
URL: https://github.com/apache/incubator-shardingsphere/issues/4031
This is an automated message from the Apache Git Service.
To respond
terrymanu commented on issue #4180: Inline strategy cannot support range
sharding
URL:
https://github.com/apache/incubator-shardingsphere/issues/4180#issuecomment-582877391
No, Inline strategy cannot support range sharding.
More details please reference document and example
coveralls commented on issue #4186: modify createInSegment() for select
URL:
https://github.com/apache/incubator-shardingsphere/pull/4186#issuecomment-582874056
## Pull Request Test Coverage Report for [Build
1625](https://coveralls.io/builds/28564985)
* **0** of **24**
tristaZero closed issue #4178: Only run literal SQL parser test cases if
parameter absent
URL: https://github.com/apache/incubator-shardingsphere/issues/4178
This is an automated message from the Apache Git Service.
To
tristaZero merged pull request #4189: Fixes #4178
URL: https://github.com/apache/incubator-shardingsphere/pull/4189
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
xiyelife opened a new issue #4188: sharding +encrypt rewrite for SQL `select
with `OR` and `AND`` cause StringIndexOutOfBoundsException.
URL: https://github.com/apache/incubator-shardingsphere/issues/4188
### Which version of ShardingSphere did you use?
4.0.0
### Which project did
tristaZero opened a new pull request #4186: modify createInSegment() for select
URL: https://github.com/apache/incubator-shardingsphere/pull/4186
#3914
This is an automated message from the Apache Git Service.
To respond
tuohai666 merged pull request #4185: Rename result variable
URL: https://github.com/apache/incubator-shardingsphere/pull/4185
This is an automated message from the Apache Git Service.
To respond to the message, please log on
tuohai666 merged pull request #4184: Call setWasNull in
GroupByStreamMergedResult
URL: https://github.com/apache/incubator-shardingsphere/pull/4184
This is an automated message from the Apache Git Service.
To respond to the
coveralls commented on issue #4184: Call setWasNull in GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4184#issuecomment-582826846
## Pull Request Test Coverage Report for [Build
9341](https://coveralls.io/builds/28562988)
* **6** of **6**
plazmdk opened a new pull request #4185: Rename result variable
URL: https://github.com/apache/incubator-shardingsphere/pull/4185
Changes proposed in this pull request:
- Fixed naming issue in
https://github.com/apache/incubator-shardingsphere/pull/4173
tuohai666 commented on issue #4173: Call setWasNull in GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#issuecomment-582806703
You are right, the return value should named result.
plazmdk edited a comment on issue #4173: Call setWasNull in
GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#issuecomment-582804090
I believe the code convensions are followed now - and another pr have been
made for master
Unless I
plazmdk commented on issue #4173: Call setWasNull in GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#issuecomment-582804090
I believe the code convensions are followed now - and another pr have been
made for master
plazmdk opened a new pull request #4184: Call setWasNull in
GroupByStreamMergedResult
URL: https://github.com/apache/incubator-shardingsphere/pull/4184
Changes proposed in this pull request:
- Set wasNull field which had wrong value when doing a LEFT JOIN FETCH via
Hibernate resulting
avalon566 merged pull request #4183: Fix can't find job error
URL: https://github.com/apache/incubator-shardingsphere/pull/4183
This is an automated message from the Apache Git Service.
To respond to the message, please log
KomachiSion opened a new pull request #4183: Fix can't find job error
URL: https://github.com/apache/incubator-shardingsphere/pull/4183
This is an automated message from the Apache Git Service.
To respond to the message,
coveralls edited a comment on issue #4181: modify createInSegment() for select
URL:
https://github.com/apache/incubator-shardingsphere/pull/4181#issuecomment-582790749
## Pull Request Test Coverage Report for [Build
1622](https://coveralls.io/builds/28561887)
* **0** of **9**
avalon566 merged pull request #4181: modify createInSegment() for select
URL: https://github.com/apache/incubator-shardingsphere/pull/4181
This is an automated message from the Apache Git Service.
To respond to the message,
tuohai666 commented on issue #4173: Call setWasNull in GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#issuecomment-582791411
@plazmdk You are right. GroupByStreamMergedResult overwrite the getValue()
method but miss set the wasNull value.
coveralls commented on issue #4181: modify createInSegment() for select
URL:
https://github.com/apache/incubator-shardingsphere/pull/4181#issuecomment-582790749
## Pull Request Test Coverage Report for [Build
9338](https://coveralls.io/builds/28561623)
* **0** of **9** **(0.0%)**
tuohai666 commented on a change in pull request #4173: Call setWasNull in
GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#discussion_r375692472
##
File path:
tuohai666 commented on a change in pull request #4173: Call setWasNull in
GroupByStreamMergedResult
URL:
https://github.com/apache/incubator-shardingsphere/pull/4173#discussion_r375692251
##
File path:
KomachiSion commented on issue #4180: Inline strategy cannot support range
sharding
URL:
https://github.com/apache/incubator-shardingsphere/issues/4180#issuecomment-582785368
You should code a `RangeShardingAlgorithm` and configure it as your range
algorithm.
example like :
succeo commented on issue #4180: Inline strategy cannot support range sharding
URL:
https://github.com/apache/incubator-shardingsphere/issues/4180#issuecomment-582782882
config:
shardingColumn: f_id
If you have a query for the range of columns in a separate table, how will
it
72 matches
Mail list logo
