Skip to content

expressions,tidb: unary handes bool type#161

Merged
qiuyesuifeng merged 1 commit intomasterfrom
siddontang/fix-unary-bool
Sep 15, 2015
Merged

expressions,tidb: unary handes bool type#161
qiuyesuifeng merged 1 commit intomasterfrom
siddontang/fix-unary-bool

Conversation

@siddontang
Copy link
Member

fix select + (1 > 0), - (1 > 0)

@qiuyesuifeng
Copy link
Member

LGTM

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have bool data in tidb?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some comparison expressions return bool.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will change these later. but now,we must support bool.

@shenli
Copy link
Member

shenli commented Sep 15, 2015

LGTM

qiuyesuifeng added a commit that referenced this pull request Sep 15, 2015
expressions,tidb: unary handes bool type
@qiuyesuifeng qiuyesuifeng merged commit 9d24f7e into master Sep 15, 2015
@qiuyesuifeng qiuyesuifeng deleted the siddontang/fix-unary-bool branch September 15, 2015 09:48
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
* tests: fix existing test failure

* mydump: fixed conversion of integers into bits

We need to create a special branch for integers, since casting 123 and
'123' into BIT type behave differently.

Also fixed handling of 0x/0b bit strings since Ragel doesn't recognize
'+' in a regex -_-.

* mydump: store description of `token` in an array instead of switch cases

* tests: test behavior of integers for ENUM and SET types as well
YuJuncen pushed a commit to YuJuncen/tidb that referenced this pull request Apr 23, 2021
* *: support tls

* move tikv.driver to glue

* fix comments
xhebox pushed a commit to xhebox/tidb that referenced this pull request Sep 28, 2021
xhebox pushed a commit to xhebox/tidb that referenced this pull request Oct 8, 2021
ti-chi-bot pushed a commit that referenced this pull request Oct 9, 2021
nolouch pushed a commit to tidblabs/tidb that referenced this pull request Jan 6, 2023
…ap#144 into keyspace on release 6.4  (pingcap#161)

* Merge PR pingcap#78,pingcap#79,pingcap#82,pingcap#83,pingcap#86,pingcap#144 into keyspace on release 6.4

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

* make check

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

* make check

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

* fix error

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

* fix comment

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

* make check

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>

Signed-off-by: zeminzhou <zhouzemin@pingcap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants