mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-21 15:12:02 +00:00
Fix build and style errors. Update test.
This commit is contained in:
parent
6464a1a902
commit
697d5b07ce
@ -27,14 +27,12 @@ void JSONAsStringRowInputFormat::readJSONObject(IColumn & column)
|
||||
{
|
||||
PeekableReadBufferCheckpoint checkpoint{buf};
|
||||
size_t balance = 0;
|
||||
size_t object_size = 0;
|
||||
bool quotes = false;
|
||||
|
||||
if (*buf.position() != '{')
|
||||
throw Exception("JSON object must begin with '{'.", ErrorCodes::INCORRECT_DATA);
|
||||
|
||||
++buf.position();
|
||||
++object_size;
|
||||
++balance;
|
||||
|
||||
char * pos;
|
||||
@ -47,7 +45,6 @@ void JSONAsStringRowInputFormat::readJSONObject(IColumn & column)
|
||||
if (quotes)
|
||||
{
|
||||
pos = find_first_symbols<'"', '\\'>(buf.position(), buf.buffer().end());
|
||||
object_size += pos - buf.position();
|
||||
buf.position() = pos;
|
||||
if (buf.position() == buf.buffer().end())
|
||||
continue;
|
||||
@ -55,23 +52,19 @@ void JSONAsStringRowInputFormat::readJSONObject(IColumn & column)
|
||||
{
|
||||
quotes = false;
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
else if (*buf.position() == '\\')
|
||||
{
|
||||
++buf.position();
|
||||
++object_size;
|
||||
if (!buf.eof())
|
||||
{
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
pos = find_first_symbols<'"', '{', '}', '\\'>(buf.position(), buf.buffer().end());
|
||||
object_size += pos - buf.position();
|
||||
buf.position() = pos;
|
||||
if (buf.position() == buf.buffer().end())
|
||||
continue;
|
||||
@ -79,36 +72,32 @@ void JSONAsStringRowInputFormat::readJSONObject(IColumn & column)
|
||||
{
|
||||
++balance;
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
else if (*buf.position() == '}')
|
||||
{
|
||||
--balance;
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
else if (*buf.position() == '\\')
|
||||
{
|
||||
++buf.position();
|
||||
++object_size;
|
||||
if (!buf.eof())
|
||||
{
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
}
|
||||
else if (*buf.position() == '"')
|
||||
{
|
||||
quotes = true;
|
||||
++buf.position();
|
||||
++object_size;
|
||||
}
|
||||
}
|
||||
}
|
||||
buf.makeContinuousMemoryFromCheckpointToPos();
|
||||
char * end = buf.position();
|
||||
buf.rollbackToCheckpoint();
|
||||
column.insertData(buf.position(), object_size);
|
||||
buf.position() += object_size;
|
||||
column.insertData(buf.position(), end - buf.position());
|
||||
buf.position() = end;
|
||||
}
|
||||
|
||||
bool JSONAsStringRowInputFormat::readRow(MutableColumns & columns, RowReadExtension &)
|
||||
@ -123,10 +112,7 @@ bool JSONAsStringRowInputFormat::readRow(MutableColumns & columns, RowReadExtens
|
||||
++buf.position();
|
||||
skipWhitespaceIfAny(buf);
|
||||
|
||||
if (buf.eof())
|
||||
return false;
|
||||
|
||||
return true;
|
||||
return !buf.eof();
|
||||
}
|
||||
|
||||
void registerInputFormatProcessorJSONAsString(FormatFactory & factory)
|
||||
|
@ -1,3 +1,3 @@
|
||||
{"id" : 1, "date" : "01.01.2020", "string" : "123{{{\\"\\\\", "array" : [1, 2, 3], "map": {"a" : 1, "b" : 2, "c" : 3}}
|
||||
{"id" : 2, "date" : "01.02.2020", "string" : "{another\\" string}}", "array" : [3, 2, 1], "map" : {"z" : 1, "y" : 2, "x" : 3}}
|
||||
{"id" : 3, "date" : "01.03.2020", "string" : "one more string", "array" : [3,1,2], "map" : {"{" : 1, "}}" : 2}}
|
||||
{\n "id" : 1,\n "date" : "01.01.2020",\n "string" : "123{{{\\"\\\\",\n "array" : [1, 2, 3],\n "map": {\n "a" : 1,\n "b" : 2,\n "c" : 3\n }\n}
|
||||
{\n "id" : 2,\n "date" : "01.02.2020",\n "string" : "{another\\"\n string}}",\n "array" : [3, 2, 1],\n "map" : {\n "z" : 1,\n "y" : 2, \n "x" : 3\n }\n}
|
||||
{\n "id" : 3, \n "date" : "01.03.2020", \n "string" : "one more string", \n "array" : [3,1,2], \n "map" : {\n "{" : 1, \n "}}" : 2\n }\n}
|
||||
|
47
tests/queries/0_stateless/01232_json_as_string_format.sh
Executable file
47
tests/queries/0_stateless/01232_json_as_string_format.sh
Executable file
@ -0,0 +1,47 @@
|
||||
#!/usr/bin/env bash
|
||||
|
||||
CURDIR=$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)
|
||||
. $CURDIR/../shell_config.sh
|
||||
|
||||
$CLICKHOUSE_CLIENT --query="DROP TABLE IF EXISTS json_as_string";
|
||||
|
||||
$CLICKHOUSE_CLIENT --query="CREATE TABLE json_as_string (field String) ENGINE = Memory";
|
||||
|
||||
echo '
|
||||
{
|
||||
"id" : 1,
|
||||
"date" : "01.01.2020",
|
||||
"string" : "123{{{\"\\",
|
||||
"array" : [1, 2, 3],
|
||||
"map": {
|
||||
"a" : 1,
|
||||
"b" : 2,
|
||||
"c" : 3
|
||||
}
|
||||
},
|
||||
{
|
||||
"id" : 2,
|
||||
"date" : "01.02.2020",
|
||||
"string" : "{another\"
|
||||
string}}",
|
||||
"array" : [3, 2, 1],
|
||||
"map" : {
|
||||
"z" : 1,
|
||||
"y" : 2,
|
||||
"x" : 3
|
||||
}
|
||||
}
|
||||
{
|
||||
"id" : 3,
|
||||
"date" : "01.03.2020",
|
||||
"string" : "one more string",
|
||||
"array" : [3,1,2],
|
||||
"map" : {
|
||||
"{" : 1,
|
||||
"}}" : 2
|
||||
}
|
||||
}' | $CLICKHOUSE_CLIENT --query="INSERT INTO json_as_string FORMAT JSONAsString";
|
||||
|
||||
$CLICKHOUSE_CLIENT --query="SELECT * FROM json_as_string";
|
||||
$CLICKHOUSE_CLIENT --query="DROP TABLE json_as_string"
|
||||
|
@ -1,7 +0,0 @@
|
||||
DROP TABLE IF EXISTS json_as_string;
|
||||
CREATE TABLE json_as_string (field String) ENGINE = Memory;
|
||||
|
||||
INSERT INTO json_as_string FORMAT JSONAsString {"id" : 1, "date" : "01.01.2020", "string" : "123{{{\"\\", "array" : [1, 2, 3], "map": {"a" : 1, "b" : 2, "c" : 3}} , {"id" : 2, "date" : "01.02.2020", "string" : "{another\" string}}", "array" : [3, 2, 1], "map" : {"z" : 1, "y" : 2, "x" : 3}} {"id" : 3, "date" : "01.03.2020", "string" : "one more string", "array" : [3,1,2], "map" : {"{" : 1, "}}" : 2}}
|
||||
|
||||
SELECT * FROM json_as_string;
|
||||
DROP TABLE json_as_string;
|
Loading…
Reference in New Issue
Block a user