Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | minor cleanup | Arne H Juul | 2022-03-29 | 1 | -24/+1 | |
|/ | | | | * simplify and cleanup some outdated code and comments | |||||
* | always use GeoPosType for position structs | Arne H Juul | 2022-03-28 | 1 | -3/+4 | |
| | | | | * should not cause any changes in observed behavior | |||||
* | Handle unaligned IdString reads in a well-defined manner | Tor Brede Vekterli | 2022-03-28 | 1 | -2/+16 | |
| | | | | | | Should compile down to effectively the same instructions on x86-64, and might save us some headaches on architectures that are less lenient with data alignment for reads. | |||||
* | Avoid code duplication. | Henning Baldersheim | 2022-03-28 | 1 | -2/+1 | |
| | ||||||
* | Use both lvalue and rvalue specifier to avoid explicit std::move() | Henning Baldersheim | 2022-03-28 | 5 | -86/+94 | |
| | ||||||
* | Avoid the need for clone by using unique_ptr. | Henning Baldersheim | 2022-03-28 | 21 | -169/+141 | |
| | ||||||
* | GC unused code | Henning Baldersheim | 2022-03-28 | 1 | -1/+0 | |
| | ||||||
* | Avoid need to copy/clone FieldUpdate | Henning Baldersheim | 2022-03-27 | 7 | -114/+112 | |
| | ||||||
* | Avoid multiple inheritance. | Henning Baldersheim | 2022-03-27 | 10 | -28/+45 | |
| | ||||||
* | fix two getDataType calls | Arne H Juul | 2022-03-24 | 1 | -2/+2 | |
| | ||||||
* | mark getDataType(String) as deprecated | Arne H Juul | 2022-03-24 | 4 | -13/+30 | |
| | | | | | * internal code can use getDataTypeInternal when necessary * external code should always get their data types in other ways | |||||
* | avoid DocumentTypeManager.getDataType(name) | Arne H Juul | 2022-03-24 | 6 | -28/+36 | |
| | ||||||
* | Merge pull request #21804 from vespa-engine/arnej/split-get-declare-struct | Arne H Juul | 2022-03-24 | 3 | -2/+41 | |
|\ | | | | | split into two methods (review feedback) | |||||
| * | split into two methods (review feedback) | Arne H Juul | 2022-03-24 | 3 | -2/+41 | |
| | | ||||||
* | | expand tabs and reindent | Arne H Juul | 2022-03-24 | 12 | -385/+385 | |
|/ | ||||||
* | just use data type known by the annotation type | Arne H Juul | 2022-03-23 | 1 | -2/+3 | |
| | ||||||
* | Abort when something unexpected happens. | Henning Baldersheim | 2022-03-22 | 1 | -1/+1 | |
| | ||||||
* | GC unused hasChanged concept. Only done on StructFieldValue as we never ↵ | Henning Baldersheim | 2022-03-18 | 29 | -245/+34 | |
| | | | | reference internal data. | |||||
* | Implement className on document::FieldValue | Henning Baldersheim | 2022-03-17 | 14 | -26/+65 | |
| | ||||||
* | Add explicit and GC unused getAddZeroTerm | Henning Baldersheim | 2022-03-17 | 5 | -19/+13 | |
| | ||||||
* | Add noexcept | Henning Baldersheim | 2022-03-17 | 1 | -10/+10 | |
| | ||||||
* | Avoid requiring identifiable for FieldValue | Henning Baldersheim | 2022-03-16 | 74 | -354/+271 | |
| | ||||||
* | Add basic support for BoolFieldValues in C++ document selection impl | Tor Brede Vekterli | 2022-03-15 | 3 | -11/+36 | |
| | ||||||
* | Add comparison support for BoolFieldValues in Java document selection impl | Tor Brede Vekterli | 2022-03-15 | 2 | -0/+20 | |
| | | | | | | Field _value_ itself is already extracted via field path iteration, but since bool fields do not inherit from an existing numeric field value type we need to explicitly handle the bool type when evaluating AST comparison nodes. | |||||
* | GC unused convenience methods. | Henning Baldersheim | 2022-03-15 | 3 | -14/+2 | |
| | ||||||
* | add getDeclaredStructType in DocumentType | Arne H Juul | 2022-03-13 | 5 | -3/+170 | |
| | | | | | * getType() in DocumentTypeManager will be deprecated soon, this will be the recommended replacement. | |||||
* | even TemporarySDField should be connected to a "repo" | Arne H Juul | 2022-03-11 | 1 | -0/+1 | |
| | ||||||
* | Move BucketIdListResult | Henning Baldersheim | 2022-03-09 | 3 | -8/+7 | |
| | ||||||
* | surely the TAG type is a tag type | Arne H Juul | 2022-03-09 | 1 | -1/+3 | |
| | ||||||
* | Merge pull request #21598 from vespa-engine/toregge/avoid-ambiguous-overload | Henning Baldersheim | 2022-03-08 | 1 | -1/+1 | |
|\ | | | | | Avoid ambiguous overload. | |||||
| * | Avoid ambiguous overload. | Tor Egge | 2022-03-08 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #21586 from vespa-engine/balder/gc-unused-threadpool | Henning Baldersheim | 2022-03-08 | 1 | -1/+0 | |
|\ \ | | | | | | | Balder/gc unused threadpool | |||||
| * | | Get VESPA_DLL_LOCAL from vespa/fastos/types.h | Henning Baldersheim | 2022-03-07 | 1 | -1/+0 | |
| |/ | ||||||
* / | deprecate useless constructor | Arne H Juul | 2022-03-08 | 2 | -10/+19 | |
|/ | | | | | | * always ignored non-null typeName * the intention was probably something else * was not used anywhere | |||||
* | Merge pull request #21580 from vespa-engine/arnej/verify-namecollision-fails | Jon Bratseth | 2022-03-07 | 1 | -3/+8 | |
|\ | | | | | struct/document name collision fails | |||||
| * | struct/document name collision fails | Arne H Juul | 2022-03-07 | 1 | -3/+8 | |
| | | | | | | | | | | * the unit tests claims that this works, and seems to verify that. But the generated config isn't valid. | |||||
* | | Use corrcet type for short. | Henning Baldersheim | 2022-03-07 | 1 | -1/+1 | |
| | | ||||||
* | | Use XXFieldValue::make over std::make_unique | Henning Baldersheim | 2022-03-07 | 15 | -55/+54 | |
| | | ||||||
* | | No reuse, so just use return value optimisation. | Henning Baldersheim | 2022-03-07 | 2 | -12/+9 | |
| | | ||||||
* | | GC unused non-const iterate interface. | Henning Baldersheim | 2022-03-07 | 2 | -15/+7 | |
| | | ||||||
* | | GC redundant convenience wasy of assigning primitive field values. | Henning Baldersheim | 2022-03-07 | 23 | -303/+78 | |
|/ | ||||||
* | As a first step to reduce code complexity around FieldValue move test-only ↵ | Henning Baldersheim | 2022-03-05 | 9 | -124/+115 | |
| | | | | | | convenience methods out to test-only helpers. | |||||
* | Remove clone from DataType. | Henning Baldersheim | 2022-03-04 | 27 | -120/+61 | |
| | ||||||
* | Remove identifiable from document::DataType | Henning Baldersheim | 2022-03-04 | 47 | -421/+294 | |
| | | | | Also remove the virtual == operator and the need for default constructor. | |||||
* | Reduce use of Identifiable for document::DatatType | Henning Baldersheim | 2022-03-03 | 43 | -129/+162 | |
| | ||||||
* | GC cloneable | Henning Baldersheim | 2022-03-03 | 6 | -17/+10 | |
| | ||||||
* | Explicitly precompute GlobalId for reference values to avoid lazy init race | Tor Brede Vekterli | 2022-03-03 | 2 | -1/+4 | |
| | | | | | | | | | | | | | This is a workaround for `DocumentId::getGlobalId()`'s lazy GID computation currently not being thread safe in a `const` context. The following race was previously possible: * Attribute writer thread calls `DocumentId::getGlobalId` when applying the reference value update to a reference attribute. Writes GID and flag. * Shared executor thread calls `DocumentId::operator=` with the same ID instance as the rhs argument when applying the reference value update to the doc store. Causes a read race with the above write. | |||||
* | Expect updated error message | Jon Bratseth | 2022-03-01 | 1 | -2/+1 | |
| | ||||||
* | No functional changes | Jon Bratseth | 2022-03-01 | 3 | -23/+16 | |
| | ||||||
* | Add noexcept specifiers. | Tor Egge | 2022-02-28 | 2 | -3/+3 | |
| |