Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • Kea Kea
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 594
    • Issues 594
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 61
    • Merge requests 61
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • ISC Open Source ProjectsISC Open Source Projects
  • KeaKea
  • Merge requests
  • !1529

kea-admin lease-upload mysql

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Andrei Pavel requested to merge 2039-kea-admin-lease-upload-mysql into master Dec 20, 2021
  • Overview 10
  • Commits 5
  • Pipelines 0
  • Changes 9

Part of #2039 (closed).

upload seems like the best antonym to dump that shows direction.

  • a2aa65f7 kea-admin lease-upload mysql
    • This approach tries to do as little work as possible in shell and offload everything as quickly as possible to SQL. Since there is no procedure can accept any input at least not without quotes around it, the only thing kea-admin does is put quotes around values of columns that are varchars or around values that need to be further processed in SQL.
    • This approach does not assume a fixed position of columns in the CSV file, but searches for it every time when it needs to. It only needs to do that when stringifying for the reason above. I don't necessarily want it this way so I can put static columns in there if the reviewer wants. If performance is an issue, you should know that not the ability to dynamically search for columns is not a great burden, but rather the stringification/quoting itself, or the fact that we're not doing batch lease upload, but rather one at a time. I haven't considered batch SQL very much, because I'm not familiar with how portable it is across MySQL versions and how much of a speedup it would bring.
    • dump_type -> dhcp_version because now it's also upload type not only dump type
  • f6a9a095 kea-admin lease-upload mysql tests
    • removed some tmp_file references that weren't used anymore
  • 2fb4f192 document kea-admin lease-upload
  • 5378a406 add ChangeLog entry
Edited Jan 21, 2022 by Andrei Pavel
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 2039-kea-admin-lease-upload-mysql