summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Use correct attribute for x509 cert chainBjørn Christian Seime2017-12-041-1/+1
| | | | | | The JDisc attribute will not return the cert chain here as the JDisc attributes are not back-propagated to the underlying Servlet request. The fix is to use the corresponding Servlet attribute.
* Merge pull request #4336 from vespa-engine/gjoranv/remove-container-dev-builderBjørn Christian Seime2017-12-048-320/+0
|\ | | | | Remove unused module container-dev-builder.
| * Remove unused module container-dev-builder.gjoranv2017-12-048-320/+0
| |
* | Merge pull request #4338 from ↵Henning Baldersheim2017-12-0491-4419/+26
|\ \ | | | | | | | | | | | | vespa-engine/vekterli/remove-deprecated-memory-manager Remove deprecated memory manager
| * | Remove unused (and not working) graphing utilitiesTor Brede Vekterli2017-12-0415-945/+0
| | |
| * | Remove memory manager component from content layerTor Brede Vekterli2017-12-0478-3474/+26
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | We already have resource utilization tracking in both MessageBus and the search core. The memory manager has never been auto-scaled based on the hardware present and adds a _lot_ of complexity without having any known instances where it has actually saved the day. Removing it also removes a mutex on the message hot path. If we need such functionality in the future, should design a lock-free solution. Cleanup
* | Merge pull request #4334 from vespa-engine/arnej/add-vespalib-metrics-7Arne H Juul2017-12-0465-27/+2409
|\ \ | |/ |/| Arnej/add vespalib metrics 7
| * move MockTickArne Juul2017-12-046-11/+10
| |
| * update metrics library after reviewsArne Juul2017-12-0432-341/+477
| | | | | | | | | | | | | | | | | | | | * use slimmer handle subclasses * new tick abstraction * gc unused time supplier * rename current bucket concept to current samples * rename TimeStamp type * add padding making old metrics visible * improve test, now with less debug printing.
| * review follow-upArne Juul2017-12-0329-182/+272
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * bug fix PointMap::operator< and use "for" loop * construct aggregators from samples only * use generation counter instead of timestamps to ensure correct ordering * add "noexcept" to signal strong exception guarantees * handle out-of-order metric type checks * ensure ticker thread is stopped in destructor * change TickerThread::stop() to only do its work once, as calling stop() twice crashed for some strange reason * refactor CurrentSamples extraction * move time-handling to tick() * ensure we do not have concurrent tick()s running * simpler time handling
| * update metrics library after reviewArne Juul2017-12-0349-559/+833
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fix PointMap comparison bug and style. * Rename ReAllocBunch to StableStore, and GC its unused lookup method. Rename method apply -> for_each * Rename CounterIncrement -> Counter::Increment and GaugeMeasurement -> Gauge::Measurement * Rename PointMapBacking -> PointMap::BackingMap * Make single-argument constructors explicit. * Do not use actual time in dummy. * Rename mergeWithMap -> mergeFromSamples, and do some refactoring of it. * Refactor opaque handle implementation into its own class * Copy strings into snapshot, do not refer to strings owned elsewhere. * Refactor _pointMaps code into new PointMapCollection class * Split aggregators into their own files, move Bucket into its own files, and rename files for CurrentSamples. * Make Counter and Gauge only hold the MetricName and supply an empty Point via default argument instead. * Split out StableStore test, and test for pointer equality. * Separate out ticker thread, and arrange for fast stop. * Use less swap. * Add totalSnapshot() facility.
| * first take on new simple metrics library.Arne Juul2017-12-0346-27/+1910
| | | | | | | | | | | | This is still a Work In Progress, the most basic public APIs should be stable now but most of the implementation is still untested and undocumented.
* | Merge pull request #4328 from ↵Tor Brede Vekterli2017-12-0412-68/+638
|\ \ | | | | | | | | | | | | vespa-engine/vekterli/add-global-distribution-transformation-utility Add global distribution config transformation utility
| * | Add utility for transforming default distribution config to globalTor Brede Vekterli2017-12-015-0/+560
| | | | | | | | | | | | | | | Global distribution config in this context means a config that places a replica of every bucket on every node in the group hierarchy.
| * | Remove no-op code and clean up DistributionTor Brede Vekterli2017-12-015-48/+58
| | | | | | | | | | | | Move group config path parsing into utility class.
| * | Less Gallic influence on namingTor Brede Vekterli2017-12-012-20/+20
| | |
* | | Merge pull request #4331 from vespa-engine/hmusum/handle-file-already-existingHenning Baldersheim2017-12-041-5/+19
|\ \ \ | | | | | | | | Handle file already exisisting
| * | | Handle file already exisistingHarald Musum2017-12-011-5/+19
| |/ / | | | | | | | | | | | | | | | When we ask several config servers for a file we might get several callbacks which all try to move a file to the destination. Do not fail if we cannot move the file because it already exists
* | | Merge pull request #4322 from ↵Henning Baldersheim2017-12-048-46/+59
|\ \ \ | | | | | | | | | | | | | | | | vespa-engine/geirst/add-more-proton-resource-limit-metrics Add metrics on how far away from disk and memory resource limits prot…
| * | | Add metrics on how far away from disk and memory resource limits proton is.Geir Storli2017-12-018-46/+59
| | | |
* | | | Merge pull request #4335 from vespa-engine/mpolden/revert-log-level-changeMorten Tokle2017-12-041-1/+1
|\ \ \ \ | | | | | | | | | | Revert log level change
| * | | | Revert "Reduce to FINER"Martin Polden2017-12-041-1/+1
| | | | | | | | | | | | | | | | | | | | This reverts commit 833c8f6b6c6e0e611e9ed8ea6e2d7fe9f56b2c48.
* | | | | Merge pull request #4326 from vespa-engine/freva/add-transitioning-stateValerij Fredriksen2017-12-042-25/+59
|\ \ \ \ \ | |_|_|_|/ |/| | | | Add TRANSITIONING state in NodeAdminStateUpdater
| * | | | Add TRANSITIONING state in NodeAdminStateUpdaterValerij Fredriksen2017-12-012-25/+59
| | | | |
* | | | | Merge pull request #4332 from vespa-engine/gjoranv/split-parent3gjoranv2017-12-02110-965/+1291
|\ \ \ \ \ | | | | | | | | | | | | Gjoranv/split parent3
| * | | | | Use new version properties from container-dependency-versions.gjoranv2017-12-011-22/+22
| | | | | |
| * | | | | Use no_aop classifier for guice.gjoranv2017-12-013-0/+3
| | | | | |
| * | | | | Install new parent poms.gjoranv2017-12-011-1/+9
| | | | | |
| * | | | | Move all deps using properties from container-dep-versions upgjoranv2017-12-012-37/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | .. from parent to container-dep-versions. - The aim was to have only deps provided runtime from jdisc in container-dep-versions, but this is not possible because factorylib reads the text in parent/pom.xml (and now container-dep-versions/pom.xml) to build a pom model used to bootstrap the maven cache on factory. Hence all deps using properties declared in a pom also have to reside in the same pom.
| * | | | | Split parent + container-dependency-versions from root pom.gjoranv2017-12-01109-942/+1241
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | - Add missing dependencies so that all provided non-yahoo jars are listed in container-dependency-versions. - Add relativePath for all child poms of parent.
* | | | | Merge pull request #4329 from vespa-engine/arnej/better-test-xw-productHåvard Pettersen2017-12-013-310/+65
|\ \ \ \ \ | | | | | | | | | | | | Arnej/better test xw product
| * | | | | consolidate testsArne Juul2017-12-011-12/+4
| | | | | |
| * | | | | simplify castsArne Juul2017-12-012-6/+4
| | | | | |
| * | | | | improve testArne Juul2017-12-011-295/+60
| | |_|_|/ | |/| | |
* | | | | Merge pull request #4330 from vespa-engine/havardpe/simplify-castingArne H Juul2017-12-011-2/+2
|\ \ \ \ \ | |/ / / / |/| | | | use static cast instead of dynamic cast
| * | | | use static cast instead of dynamic castHåvard Pettersen2017-12-011-2/+2
|/ / / /
* | | | Merge pull request #4321 from ↵gjoranv2017-12-018-65/+155
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | vespa-engine/hmusum/do-not-setup-FiledistributorService-when-filedistributor-is-disabled Do not setup a service for filedistributor when it is disabled
| * | | | Do not setup a service for filedistributor when it is disabledHarald Musum2017-11-308-65/+155
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some workarounds needed to still file distribution config even when such a serice is not set up (can be simplified a lot when file distributor is removed for good)
* | | | | Merge pull request #4327 from ↵Arne H Juul2017-12-015-32/+152
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | vespa-engine/havardpe/recognize-and-replace-xw-product replace appropriate sub-expressions with dense xw product
| * | | | | replace appropriate sub-expressions with dense xw productHåvard Pettersen2017-12-015-32/+152
| | | | | |
* | | | | | Merge pull request #4324 from ↵Tor Egge2017-12-012-0/+4
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | vespa-engine/vekterli/make-getbucketlistmessage-destructor-explicit Make GetBucketListMessage destructor explicit
| * | | | | Make GetBucketListMessage destructor explicitTor Brede Vekterli2017-12-012-0/+4
| | |_|/ / | |/| | | | | | | | | | | | | Avoids inlining size warnings when compiling with debug flags
* | | | | Merge pull request #4325 from vespa-engine/arnej/add-dense-xw-productHåvard Pettersen2017-12-016-0/+500
|\ \ \ \ \ | |_|_|/ / |/| | | | implement and test XW product
| * | | | implement and test XW productArne Juul2017-11-306-0/+500
| | | | |
* | | | | Merge pull request #4323 from vespa-engine/freva/reset-wantTo-fields-on-readyMartin Polden2017-12-015-8/+62
|\ \ \ \ \ | |_|/ / / |/| | | | Reset wantTo* fields with transitioning to ready
| * | | | Reset wantTo* fields with transitioning to readyValerij Fredriksen2017-12-015-8/+62
|/ / / /
* | | | Merge pull request #4317 from vespa-engine/freva/use-https-against-configserverValerij Fredriksen2017-12-0110-150/+137
|\ \ \ \ | |_|/ / |/| | | Node-Admin: Use HTTPS against config servers
| * | | Remove sys.out in testValerij Fredriksen2017-11-302-14/+6
| | | |
| * | | Node-Admin: Use HTTPS against config serversValerij Fredriksen2017-11-3010-138/+133
| | | |
* | | | Merge pull request #4319 from vespa-engine/hmusum/remove-unused-ymon-propertyHarald Musum2017-11-3010-27/+1
|\ \ \ \ | |_|_|/ |/| | | Remove unused ymonService property