Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert ES/BOOLEAN to SQL C BIT #20

Merged
merged 2 commits into from
Sep 4, 2018
Merged

Convert ES/BOOLEAN to SQL C BIT #20

merged 2 commits into from
Sep 4, 2018

Conversation

bpintea
Copy link
Collaborator

@bpintea bpintea commented Sep 3, 2018

ES/BOOLEAN is used in results to catalog queries, like SYS TYPES (used by SQLGetTypeInfo() API). Since this is a non-ODBC standard type, some applications (like Excel) have troubles automatically converting it to the corresponding integer required by the standard.

This PR changes the handling of ES/BOOLEAN, converting it to the BIT type, meant by ODBC for booleans.

The driver still accepts the ES/BOOLEAN type from the application, though, if this is ES types-aware.

ES/BOOLEAN is used in results to catalog queries, like SYS TYPES (used
by SQLGetTypeInfo()). Since this is a non-ODBC standard type, some
applications (like Excel) have troubles converting it to the
coresponding standard integer required by the standard.

This commit changes the handling of ES/BOOLEAN, converting it to the BIT
type (used by ODBC for bools).

The driver still accepts the ES/BOOLEAN type from the application,
though, if this is ES types-aware.
- long long is on 8 bytes also on x86
@bpintea bpintea mentioned this pull request Sep 3, 2018
Closed
@bpintea bpintea changed the title Convert ES/BOOLEAN to C BIT Convert ES/BOOLEAN to SQL C BIT Sep 3, 2018
Copy link
Contributor

@droberts195 droberts195 left a comment

Choose a reason for hiding this comment

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

LGTM

@bpintea bpintea merged commit 26599dc into elastic:master Sep 4, 2018
@bpintea bpintea deleted the fix/es_boolean_to_c_bit branch September 4, 2018 10:39
@bpintea bpintea added >feature Applicable to PRs adding new functionality v6.5.0 labels May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>feature Applicable to PRs adding new functionality v6.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants