The rtpengine module is not affected by this change as it currently ignores the new column. Updated db scripts and documentations.
Is it possible for the http://kamailio.org/docs/db-tables/kamailio-db-devel.html to be regenerated after this commit? You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/539
-- Commit Summary --
* rtpengine: Add timestamp column to rtpengine table
-- File Changes --
M lib/srdb1/schema/rtpengine.xml (8) M modules/rtpengine/doc/rtpengine_admin.xml (29) M utils/kamctl/db_berkeley/kamailio/rtpengine (6) M utils/kamctl/db_sqlite/rtpengine-create.sql (1) M utils/kamctl/dbtext/kamailio/rtpengine (2) M utils/kamctl/mysql/rtpengine-create.sql (1) M utils/kamctl/oracle/rtpengine-create.sql (1) M utils/kamctl/postgres/rtpengine-create.sql (1) M utils/kamctl/xhttp_pi/rtpengine-mod (2) M utils/kamctl/xhttp_pi/rtpengine-table (1)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/539.patch https://github.com/kamailio/kamailio/pull/539.diff
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/539
Stefan Mititelu writes:
The rtpengine module is not affected by this change as it currently ignores the new column.
Then why the column is needed and at this stage of 4.4 development?
-- Juha
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/539#issuecomment-195029904
Then why the column is needed and at this stage of 4.4 development?
It is useful on mySQL side, to have an idea when the node was added. Maybe in the future, rtpengine module will also display this info when 'kamctl fifo nh_show_rtpp'.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/539#issuecomment-195249444
This can be merged to master, branch 4.4 was created.
I can't have any opinion myself if this worth backporting.
--- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/539#issuecomment-196260639
Merged #539.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/539#event-596855845