Skip to content

[#1102] Add Mork Adapter support#1110

Open
marcocapozzoli wants to merge 3 commits into
masterfrom
masc/1102-adapterdb-mork
Open

[#1102] Add Mork Adapter support#1110
marcocapozzoli wants to merge 3 commits into
masterfrom
masc/1102-adapterdb-mork

Conversation

@marcocapozzoli
Copy link
Copy Markdown
Collaborator

  • Extend AdapterDbType to include Mork.
  • Update AdapterDB initialization to support Mork.
  • Refactor ContextLoader methods and rename for clarity.
  • Implement MorkConnection and MorkWrapper classes.
  • Add MorkMappingStrategy for future mapping tasks.
  • Update DatabaseWrapper to create Mork2AtomsMapper.
  • Create BUILD files for Mork integration.

- Extend AdapterDbType to include Mork.
- Update AdapterDB initialization to support Mork.
- Refactor ContextLoader methods and rename for clarity.
- Implement MorkConnection and MorkWrapper classes.
- Add MorkMappingStrategy for future mapping tasks.
- Update DatabaseWrapper to create Mork2AtomsMapper.
- Create BUILD files for Mork integration.
Copy link
Copy Markdown
Contributor

@andre-senna andre-senna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed in the projects's MM channel

Comment thread src/atomdb/adapterdb/AdapterDB.h Outdated
if (value == "mork") return AdapterDbType::Mork;
RAISE_ERROR("Unsupported adapterdb.type: " + value);
}
};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
};
}

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