Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add fixes and improvements to compaction jobs and to tasks app #6276

Merged

Conversation

harenson
Copy link
Contributor

@harenson harenson commented Jan 28, 2020

  • Move auto compaction code from kz_tasks_trigger module to kt_compactor module.
  • Make kz_tasks_trigger gen_server work as a local gen_server (not using kz_globals).
  • Fix compaction job's (csv tasks/auto compaction) call_id by prefixing it with YYYYMM- so it can be retrieved in later months.
  • Remove kz_globals usage from tasks app.

@harenson harenson force-pushed the move-auto-compaction-stuff-out-of-kz_tasks_trigger branch 2 times, most recently from f9132e4 to 491a406 Compare January 29, 2020 23:40
@harenson harenson changed the title Move auto compaction stuff out of kz tasks trigger Add fixes and improvements to compaction jobs and to tasks app Jan 31, 2020
@harenson harenson marked this pull request as ready for review January 31, 2020 18:32
@harenson harenson force-pushed the move-auto-compaction-stuff-out-of-kz_tasks_trigger branch from b9215a5 to 3af40f7 Compare February 5, 2020 21:40
jamesaimonetti pushed a commit that referenced this pull request Feb 29, 2020
- Move auto compaction code from kz_tasks_trigger module to
kt_compactor module.

- Make kz_tasks_trigger gen_server work as a local gen_server (not
using kz_globals).

- Fix compaction job's (csv tasks/auto compaction) call_id by
prefixing it with `YYYYMM-` so it can be retrieved in later months.

- Remove kz_globals usage from tasks app.
@jamesaimonetti jamesaimonetti force-pushed the move-auto-compaction-stuff-out-of-kz_tasks_trigger branch from 3af40f7 to 3f9ccfd Compare February 29, 2020 00:51
- Move auto compaction code from kz_tasks_trigger module to
kt_compactor module.

- Make kz_tasks_trigger gen_server work as a local gen_server (not
using kz_globals).

- Fix compaction job's (csv tasks/auto compaction) call_id by
prefixing it with `YYYYMM-` so it can be retrieved in later months.

- Remove kz_globals usage from tasks app.
@jamesaimonetti jamesaimonetti force-pushed the move-auto-compaction-stuff-out-of-kz_tasks_trigger branch from 3f9ccfd to f21b4e9 Compare February 29, 2020 00:52
@jamesaimonetti jamesaimonetti merged commit f21b4e9 into master Feb 29, 2020
@jamesaimonetti jamesaimonetti deleted the move-auto-compaction-stuff-out-of-kz_tasks_trigger branch February 29, 2020 00:52
harenson added a commit that referenced this pull request Apr 17, 2020
- Move auto compaction code from kz_tasks_trigger module to
kt_compactor module.

- Make kz_tasks_trigger gen_server work as a local gen_server (not
using kz_globals).

- Fix compaction job's (csv tasks/auto compaction) call_id by
prefixing it with `YYYYMM-` so it can be retrieved in later months.

- Remove kz_globals usage from tasks app.
harenson added a commit that referenced this pull request Apr 17, 2020
- Move auto compaction code from kz_tasks_trigger module to
kt_compactor module.

- Make kz_tasks_trigger gen_server work as a local gen_server (not
using kz_globals).

- Fix compaction job's (csv tasks/auto compaction) call_id by
prefixing it with `YYYYMM-` so it can be retrieved in later months.

- Remove kz_globals usage from tasks app.
harenson added a commit that referenced this pull request Apr 17, 2020
- Move auto compaction code from kz_tasks_trigger module to
kt_compactor module.

- Make kz_tasks_trigger gen_server work as a local gen_server (not
using kz_globals).

- Fix compaction job's (csv tasks/auto compaction) call_id by
prefixing it with `YYYYMM-` so it can be retrieved in later months.

- Remove kz_globals usage from tasks app.
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.

2 participants