8000 Feature/internal issue #672 by Dronplane · Pull Request #11370 · arangodb/arangodb · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@Dronplane
Copy link

Adds primarySortCompression property to view definition to control compression of primarySort column. Possible values are "lz4"(backward compatible default) and "none" (store without compression)

Changes definition of storedValues property. Now array could contain objects with fields:
{
field: <<field definiton, string or array of strings>>
compression: <> Possible values are "lz4"(backward compatible default) and "none" (store without compression)
}

Fix for internal issue #690 - definition of storedValues excluded from view properties output.

Fixed JACCARD function to return value 1 for two empty arguments.

Verified by js and gtest

@Dronplane Dronplane requested a review from gnusi April 1, 2020 17:02

auto const field = slice.get(fieldName);
mask->_storedValues = field.isArray();
mask->_storedValues = !field.isNone();
Copy link
Contributor

Choose a reason for hiding this comment

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

why not field.isArray()?

Copy link
Author

Choose a reason for hiding this comment

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

Because if somebody will supply string or object, this will be silently dropped. And one will not know there is nothing stored. It is more important now, as storedValues are excluded from properties output.

@Dronplane
Copy link
Author

Andrei Lobov added 2 commits April 3, 2020 10:53
@Dronplane
Copy link
Author

@Dronplane Dronplane requested a review from gnusi April 3, 2020 09:34
@Dronplane Dronplane marked this pull request as ready for review April 3, 2020 09:34
}
auto const& mangled_name = type->name();
TRI_ASSERT(!mangled_name.empty());
auto demandled_start = mangled_name.end() - 1;
Copy link
Contributor

Choose a reason for hiding this comment

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

demandled -> demangled

@gnusi gnusi merged commit bce6f8f into devel Apr 3, 2020
@gnusi gnusi deleted the feature/internal-issue-#672 branch April 3, 2020 10:51
ObiWahn added a commit that referenced this pull request Apr 6, 2020
…ture/utf-8-validation

* 'devel' of https://github.com/arangodb/arangodb: (21 commits)
  Bug fix/headers cleanup (#11391)
  Feature/internal issue #672 (#11370)
  GraphNodes now copy the graph when cloned (#11345)
  fix lame compile error
  Feature/aql subquery execution block impl execute implementation harvesting (#11349)
  remove obsolete recoveryData
  Feature/dismantle mmfiles (#11354)
  Fix an agency supervision bug. (#11356)
  (mostly) restore pre-3.7 API behavior (#11364)
  validation: AQL functions (#11327)
  Check MSVC_VERSION instead of CMAKE_GENERATOR (#11351)
  added sleep time (lousy fix)
  Feature/aql interleave function (#11352)
  cheapify IN lookups on unsorted arrays (#11342)
  Bug fix/fix msvc2019 build (#11052)
  Allow easier removal of validation rules. (#11346)
  upgrade RocksDB (#11308)
  Introduce more type-safe identifiers (#11270)
  Bug fix/schema validation return code (#11341)
  Fix explainer output when restricting collections (#11338)
  ...
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

0