{"push_id":1241902540,"size":23,"distinct_size":23,"ref":"refs/heads/master","head":"036a1265f2b55cd731e05edc9796487203fa13d1","before":"9163673d330c8575996c65c1cc469bd6dc3f4bb4","commits":[{"sha":"f0786b32f2c0ede67ca025dc3a29b3604f4e2744","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"logmsg: move handle fixup code to a module of its own\n\nInstead of fixing up the handles in an NVTable in functions pretty close\nto the NVTable deserialization code, do that in a clearly LogMessage\nspecific module.\n\nWe were depending on LogMessage concepts a lot (stuff like how an .SDATA\nname-value pair looks like and the fact that there are structured\ndata name-value pairs where we keep the ordering), so it makes sense\nto put this at a higher layer. It still pokes into NVTable internals a\nlot, so clearly it is pretty tightly coupled with both LogMessage and\nNVTable.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/f0786b32f2c0ede67ca025dc3a29b3604f4e2744"},{"sha":"7b769695a0e67709ed0dc1e3a8bcd8407e9ac333","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"nvtable: handle unknown handles in nv_registry_get_handle_flags\n\nThis is needed because in subsequent patches, we add looking up handles that\nhave not yet been registered during deserialization, whereas this function\nhas only been called with registered handles before.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/7b769695a0e67709ed0dc1e3a8bcd8407e9ac333"},{"sha":"17686f1e8dd2c3118043e04297720dadd5347880","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"logmsg-serialize-fixup: try to reuse the same NVHandle when fixing up handles\n\nIn the deserialization path, we always looked up the new handle for the\nname being deserialized. This is not always needed, especially when the\nsame syslog-ng instance is reading back the message (which often happens\nin the case of a disk queue).\n\nThis patch whether the old handle still references the same name, which\nwe can do without going through the NVRegistry hash table. (e.g.\nO(1) instead of O(logN)). Seems to increase deserialization speed from\n150k to 180k/sec, at least with -O2. Interestingly it gets slower\nwithout O2.\n\nBut now as we do have information about wheter any of the handles changed\nat all, we can also remove a lot of processing, namely:\n  - updating of the SDATA array\n  - sorting the index\n\nThis boosts performance from 180k to 270k/sec.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/17686f1e8dd2c3118043e04297720dadd5347880"},{"sha":"3c3ed35f45620d75fb2533e095de16ca3b4c4fc6","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"serialize: make typed read/write functions to be inline\n\nSome of the serialize read/write functions are called a lot and by\nhiding them in the .c file, the compiler has to embed real call\ninstructions, which clearly shows in the profile. With this change\nI could increase LogMessage deserialization speed from 270k to 330k with\n-O2.\n\nI decided to keep the error handling parts private, those would bloat\nthe call-sites a lot and are rarely executed anyway (and\nintroduce intrusive dependencies in the header).\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/3c3ed35f45620d75fb2533e095de16ca3b4c4fc6"},{"sha":"ef63c6ebdf0afe35c0d7c7a694c8a850eefa1b06","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"logmsg-serialize: don't zero initialize the sdata array\n\nThe entire array is initialized right after the allocation, no need\nto initialize it to zero.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/ef63c6ebdf0afe35c0d7c7a694c8a850eefa1b06"},{"sha":"15e3da123dfbc1162918b73cd0facc324e403144","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"logmsg: make log_msg_get_value_name() inline\n\nThis function is called a lot during deserialization and the call overhead\nwas actually visible from the profile.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/15e3da123dfbc1162918b73cd0facc324e403144"},{"sha":"8972f32f751a8dd999c492526924c5acfadcf375","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"tags-serialize: use scratch buffers in tags deserialization\n\nTags are serialized as strings and when reading them back, we used\nserialize_read_cstring() which in turn allocates a buffer for each\nstring read. If a message has a couple of tags (which it does by default),\nthis can add to the malloc load. By using scratch buffers here we eliminate\nthat.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/8972f32f751a8dd999c492526924c5acfadcf375"},{"sha":"107d554e91b4f00b748ae5ce71d9beba55512d74","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"serialize: introduce array read/write functions for uint32\n\nTo amortize the call overhead of serialize_archive_read_bytes() to these\nin one chunk and then convert the byte order later.\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/107d554e91b4f00b748ae5ce71d9beba55512d74"},{"sha":"2f040c854efe6f7c4c3f6b81ada7a8beb6144014","author":{"name":"Balazs Scheidler","email":"dcdbeec593d63c312787688b932aee771682ec3e@balabit.com"},"message":"logmsg: use array serialization functions to improve performance\n\nSigned-off-by: Balazs Scheidler <balazs.scheidler@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/2f040c854efe6f7c4c3f6b81ada7a8beb6144014"},{"sha":"51f4e421f03e87fa287dff4dcc9dc7efb89e0ec9","author":{"name":"Noémi Ványi","email":"b7ceca6d607a48e1c81adeb6db3ba7aaeebbf6df@gmail.com"},"message":"strptime-tz: FreeBSD compatibility\n\nApparently, `timezone` is a function in time.h on FreeBSD. The function\nreturns char*, not long int as it does in other operation systems.\n\nIn case of BSDs (including ones running on Apple products) now get\ntimezone offset from the `tm_gmtoff` field.\n\nSigned-off-by: Noémi Ványi <sitbackandwait@gmail.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/51f4e421f03e87fa287dff4dcc9dc7efb89e0ec9"},{"sha":"9d7298babbd83d840bf26d7852962b0dfceced08","author":{"name":"Noémi Ványi","email":"b7ceca6d607a48e1c81adeb6db3ba7aaeebbf6df@gmail.com"},"message":"dbparser: add missing sys/wait.h include\n\nOn FreeBSD WEXITSTATUS is undefined without including the wait.h file.\n\nSigned-off-by: Noémi Ványi <sitbackandwait@gmail.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/9d7298babbd83d840bf26d7852962b0dfceced08"},{"sha":"24d17aba6340b542a6d85594600577db57b76be9","author":{"name":"Noémi Ványi","email":"b7ceca6d607a48e1c81adeb6db3ba7aaeebbf6df@gmail.com"},"message":"affile: add missing include to tests\n\nSigned-off-by: Noémi Ványi <sitbackandwait@gmail.com>\nSigned-off-by: bkil-syslogng <tamas.nagy@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/24d17aba6340b542a6d85594600577db57b76be9"},{"sha":"90d4b314652a9ddc36377d21f4b62f9664dfd060","author":{"name":"Laszlo Budai","email":"67b412e917dff5cc501bd5d39700433529ca3bb4@gmail.com"},"message":"elasticsearch_v2: set default path.home for node client\n\nFixes: #1145\n\nSigned-off-by: Laszlo Budai <stentor.bgyk@gmail.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/90d4b314652a9ddc36377d21f4b62f9664dfd060"},{"sha":"68334ecdcf448787003f1f8265e675c8984c8045","author":{"name":"Laszlo Budai","email":"67b412e917dff5cc501bd5d39700433529ca3bb4@gmail.com"},"message":"add-contextual-data: build fixed on SLES-11\n\n * include missing header\n * as `g_array_set_clear_func` is only available from glib version 2.32,\n   I removed it\n\nFixes: #1144\n\nSigned-off-by: Laszlo Budai <stentor.bgyk@gmail.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/68334ecdcf448787003f1f8265e675c8984c8045"},{"sha":"16a21b8237a54e958d32d8196419374e7fe31ec5","author":{"name":"Laszlo Budai","email":"f93ee0e61047dcee8adfe14d87dc20e1ade1d8e3@balabit.com"},"message":"cfg-tree: support platforms where glib version < 2.32\n\n* use g_hash_table_replace instead of g_hash_table_add\n\nDocumentation[1] says :\n  This is a convenience function for using a GHashTable as a set.\n  It is equivalent to calling g_hash_table_replace() with key as\n  both the key and the value.\n\n[1] - https://developer.gnome.org/glib/stable/glib-Hash-Tables.html#g-hash-table-replace\n\n* use g_hash_table_lookup_extended instead of g_hash_table_contains\n\nSigned-off-by: Laszlo Budai <Laszlo.Budai@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/16a21b8237a54e958d32d8196419374e7fe31ec5"},{"sha":"fc75e5a8c21b193a32847ce3660a38e59f2bef49","author":{"name":"Laszlo Budai","email":"67b412e917dff5cc501bd5d39700433529ca3bb4@gmail.com"},"message":"elastic-v2: fixed debian packaging\n\nelastic-v2 is part of the mod-elastic package (plugin.conf is common\nfor elastic and elastic-v2 drivers).\nJest client is also added to this package.\n\nThis may be a little bit confusing, but I have my own reasons:\n * syslog-ng supports Elastisearch: not 1.7, 2.x, 5, but Elasticsearch\n * elastic and elastic-v2 exists just because the Elasticsearch Java clients\n   are not compatible between 1.7.x and 2.x versions (and the bad thing\nthat the package names are almost identical, but internal representation\nhas been changed)\n * elastic-v2 could be packaged separately, but we have a http mode extension\n   to that which is able to work with both ElasticSearch 1.x, 2.x and Elastic5\n\nSo the elastic is only for 1.7.x series, elastic-v2 could work with 2.x series\nwith node, transport and shield mode, and elastic-v2 http mode could work\nwith all versions.\n\nReferences: #1118\n\nSigned-off-by: Laszlo Budai <stentor.bgyk@gmail.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/fc75e5a8c21b193a32847ce3660a38e59f2bef49"},{"sha":"c9a7e20690e382b9662a3d40f636255418957c52","author":{"name":"Laszlo Budai","email":"f93ee0e61047dcee8adfe14d87dc20e1ade1d8e3@balabit.com"},"message":"date-parser: use timezone only on those platforms where it is available\n\nSigned-off-by: Laszlo Budai <Laszlo.Budai@balabit.com>","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/c9a7e20690e382b9662a3d40f636255418957c52"},{"sha":"03499557045af7d068bfbdbb491db3037416d8ac","author":{"name":"Noémi Ványi","email":"79232ac53823d148ba933c28ea5fd0fea46e8fee@users.noreply.github.com"},"message":"Merge pull request #1 from lbudai/f/kvch-3.8-freebsd\n\ndate-parser: use timezone only on those platforms where it is available","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/03499557045af7d068bfbdbb491db3037416d8ac"},{"sha":"4af3d03a975c73b341535d40a628c023f6eb0f29","author":{"name":"Budai Laszlo","email":"4fc05848b45c2f4d99dd8f90cc81a7f3b06fdcb3@balabit.com"},"message":"Merge pull request #1133 from balabit/f/logmsg-serialize-performance\n\nF/logmsg serialize performance","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/4af3d03a975c73b341535d40a628c023f6eb0f29"},{"sha":"09a8d254b5acc31e5c920eedd04f95d84d54f8e6","author":{"name":"Budai Laszlo","email":"4fc05848b45c2f4d99dd8f90cc81a7f3b06fdcb3@balabit.com"},"message":"Merge pull request #1141 from kvch/3.8-freebsd\n\nFreeBSD compatibility of 3.8 - fixes #1138","distinct":true,"url":"https://api.github.com/repos/bkil-syslogng/syslog-ng/commits/09a8d254b5acc31e5c920eedd04f95d84d54f8e6"}]}
{"actor":{"display_login":"bkil-syslogng"}}