Skip to content

Commit 2bcbc41

Browse files
committed
Remove CONSTANT_KEYWORD support (#265)
Remove the type, following elasticsearch/#60524. (keyword is now the umbrella type.) (cherry picked from commit a52d695)
1 parent 679c1c1 commit 2bcbc41

File tree

5 files changed

+7
-22
lines changed

5 files changed

+7
-22
lines changed

driver/connect.c

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,6 @@
6969
#define TYPE_UNSUPPORTED "UNSUPPORTED"
7070
/* 12 */
7171
#define TYPE_SCALED_FLOAT "SCALED_FLOAT"
72-
/* 16 */
73-
#define TYPE_CONSTANT_KEYWORD "CONSTANT_KEYWORD"
7472

7573
/*
7674
* intervals
@@ -2273,15 +2271,6 @@ static BOOL elastic_name2types(wstr_st *type_name,
22732271
return TRUE;
22742272
}
22752273
break;
2276-
2277-
/* 16: CONSTANT_KEYWORD */
2278-
case sizeof(TYPE_CONSTANT_KEYWORD) - 1:
2279-
if (! wmemncasecmp(type_name->str,
2280-
MK_WPTR(TYPE_CONSTANT_KEYWORD), type_name->cnt)) {
2281-
*c_sql = ES_CKEYWORD_TO_CSQL;
2282-
*sql = ES_CKEYWORD_TO_SQL;
2283-
return TRUE;
2284-
}
22852274
}
22862275

22872276
return elastic_intervals_name2types(type_name, c_sql, sql);

driver/convert.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3813,7 +3813,7 @@ SQLRETURN c2sql_boolean(esodbc_rec_st *arec, esodbc_rec_st *irec,
38133813
/*INDENT-ON*/
38143814

38153815
DBGH(stmt, "parameter (pos#%llu) converted to boolean: %d.",
3816-
(uint64_t)pos, val);
3816+
(uint64_t)pos, val);
38173817

38183818
if (val) {
38193819
memcpy(dest, JSON_VAL_TRUE, sizeof(JSON_VAL_TRUE) - /*\0*/1);
@@ -4516,7 +4516,7 @@ static SQLRETURN c2sql_str2interval(esodbc_rec_st *arec, esodbc_rec_st *irec,
45164516
octet_len = *octet_len_ptr;
45174517
if (octet_len <= 0) {
45184518
ERRH(stmt, "invalid interval buffer length: %lld.",
4519-
(int64_t)octet_len);
4519+
(int64_t)octet_len);
45204520
RET_HDIAGS(stmt, SQL_STATE_HY090);
45214521
}
45224522
}

driver/defs.h

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -414,7 +414,7 @@
414414
#define ESODBC_SQL_OBJECT 2002
415415
#define ESODBC_SQL_NESTED 2002
416416

417-
/* the SQL type ES/SQL uses for string types (KEYWORD, TEXT, CONSTANT_KEYWORD),
417+
/* the SQL type ES/SQL uses for string types (KEYWORD, TEXT),
418418
* plus IP and GEO */
419419
#define ESODBC_SQL_VARCHAR SQL_VARCHAR
420420

@@ -463,9 +463,6 @@
463463
/* 12: KEYWORD */
464464
#define ES_KEYWORD_TO_CSQL ES_WVARCHAR_CSQL
465465
#define ES_KEYWORD_TO_SQL ES_WVARCHAR_SQL
466-
/* 12: CONSTANT_KEYWORD */
467-
#define ES_CKEYWORD_TO_CSQL ES_WVARCHAR_CSQL
468-
#define ES_CKEYWORD_TO_SQL ES_WVARCHAR_SQL
469466
/* 12: TEXT */
470467
#define ES_TEXT_TO_CSQL ES_WVARCHAR_CSQL
471468
#define ES_TEXT_TO_SQL ES_WVARCHAR_SQL

driver/queries.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -187,8 +187,8 @@ void clear_resultset(esodbc_stmt_st *stmt, BOOL on_close)
187187
/* the cursor is contained entirely in the received body */
188188
assert(stmt->rset.body.str < stmt->rset.pack.cbor.curs.str); // &&
189189
assert(stmt->rset.pack.cbor.curs.str +
190-
stmt->rset.pack.cbor.curs.cnt <
191-
stmt->rset.body.str + stmt->rset.body.cnt);
190+
stmt->rset.pack.cbor.curs.cnt <
191+
stmt->rset.body.str + stmt->rset.body.cnt);
192192
}
193193
}
194194
memset(&stmt->rset, 0, sizeof(stmt->rset));
@@ -2528,7 +2528,7 @@ static esodbc_estype_st *match_es_type(esodbc_rec_st *irec)
25282528
case SQL_DOUBLE: /* DOUBLE, SCALED_FLOAT */
25292529
return dbc->max_float_type;
25302530
break;
2531-
case ES_WVARCHAR_SQL: /* CONSTANT_KEYWORD, KEYWORD, TEXT */
2531+
case ES_WVARCHAR_SQL: /* KEYWORD, TEXT */
25322532
case ES_VARCHAR_SQL: /* IP, GEO+ */
25332533
return dbc->max_varchar_type;
25342534
default:
@@ -2858,7 +2858,7 @@ static SQLRETURN convert_param_val(esodbc_rec_st *arec, esodbc_rec_st *irec,
28582858
return c2sql_number(arec, irec, pos, &min, &max, fixed, dest, len);
28592859

28602860
/* JSON string */
2861-
case ES_WVARCHAR_SQL: /* KEYWORD, TEXT, CONSTANT_KEYWORD */
2861+
case ES_WVARCHAR_SQL: /* KEYWORD, TEXT */
28622862
case ES_VARCHAR_SQL: /* IP, GEO+ */
28632863
return c2sql_varchar(arec, irec, pos, dest, len);
28642864

test/connected_dbc.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ extern "C" {
7373
[\"FLOAT\", 7, 7, \"'\", \"'\", null, 2, false, 3, false, false, false, null, 7, 7, 7, 0, 2, null],\
7474
[\"DOUBLE\", 8, 15, \"'\", \"'\", null, 2, false, 3, false, false, false, null, 15, 15, 8, 0, 2, null],\
7575
[\"SCALED_FLOAT\", 8, 15, \"'\", \"'\", null, 2, false, 3, false, false, false, null, 15, 15, 8, 0, 2, null],\
76-
[\"CONSTANT_KEYWORD\", 12, 32766, \"'\", \"'\", null, 2, true, 3, true, false, false, null, null, null, 12, 0, null, null],\
7776
[\"IP\", 12, 45, \"'\", \"'\", null, 2, false, 3, true, false, false, null, null, null, 12, 0, null, null],\
7877
[\"KEYWORD\", 12, 32766, \"'\", \"'\", null, 2, true, 3, true, false, false, null, null, null, 12, 0, null, null],\
7978
[\"TEXT\", 12, 2147483647, \"'\", \"'\", null, 2, true, 3, true, false, false, null, null, null, 12, 0, null, null],\

0 commit comments

Comments
 (0)