Skip to content
This repository has been archived by the owner on Apr 6, 2022. It is now read-only.

Feature/to cql #17

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from
Open

Feature/to cql #17

wants to merge 18 commits into from

Conversation

akondratyev
Copy link

No description provided.

@akondratyev akondratyev changed the base branch from master to develop July 11, 2017 13:41
@@ -38,13 +36,13 @@
@Inject private Hercules hercules;
@Inject private DataDriver dataDriver;
@Inject private Monitoring monitoring;
@Inject private SerializerProvider serializerProvider;
//@Inject private SerializerProvider serializerProvider;
@Inject private EntityListenerInvocationHelper listenerInvocationHelper;
Copy link
Author

Choose a reason for hiding this comment

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

поправить

.stream()
.map(v -> serializer.serialize(v, protocol()))
.collect(Collectors.toList());
}
Copy link
Author

Choose a reason for hiding this comment

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

cql не позволяет null. пустая строка не работает
cqlsh:PixAPI> select * from "AppDeploy" where key=textAsBlob('') ALLOW FILTERING;
Bad Request: Key may not be empty

надо проверить мы должны фильтровать такие значения, или заменять на что-то, чтобы cql работал (если возможно)

пустое значение пришло откуда-то из индекса

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant