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

Speedb Artifacts #64

Closed
bosmatt opened this issue Jul 24, 2022 · 3 comments · Fixed by #66
Closed

Speedb Artifacts #64

bosmatt opened this issue Jul 24, 2022 · 3 comments · Fixed by #66
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bosmatt
Copy link
Contributor

bosmatt commented Jul 24, 2022

  • Artifacts from a Speedb build (JAR, Libraries, etc) will include the Speedb name (e.g. libspeedb.a)
  • Installation packages (CMake packages, Maven artifactory, apt/yum installers, etc) will include "Speedb"
  • The Usage/Version of the library and executables will refer to Speedb
@osnatbm osnatbm assigned isaac-io and osnatbm and unassigned osnatbm Jul 25, 2022
@isaac-io
Copy link
Contributor

What about the include/ directory? Do we need to do anything about the C++/Java namespace? Is there a drop-in replacement path where the library is named rocksdb?

@osnatbm
Copy link

osnatbm commented Jul 25, 2022

  • Artifacts from a Speedb build (JAR, Libraries, etc) will include the Speedb name (e.g. libspeedb.a)
    User: Instead of linking their application against librocksdb.a, they will need to change it to link against libspeed.a. For Java, the package they use in their build/deploy scripts will change. The amount of work depends on the tool, but is generally pretty low
    Speedb: We will need update the Makefile et al to produce the appropriate binaries

  • Installation packages (CMake packages, Maven artifactory, apt/yum installers, etc) will include be "Speedb"
    User: CMake: The user would find a different package (speedb instead of rocksdb). We might be able to make the rest "just work", but I am not positive. It might mean that their CMake scripts have to refer to speedb instead of rocksdb. Maven: they would point at a different repository in the Maven artifactory. I am not exactly sure how this works but think it is straightforward.
    Speedb: We need to understand how the install process works and be sure to build/push the right artifacts.

  • The Usage/Version of the library and executables will refer to Speedb, not RocksDB
    User: No impact unless their scripts comb the values looking for strings.
    Speedb: Development/QA time to make sure it works for all executables

-Change readme and relevant user guides (getting started and maybe how to contribute)

@osnatbm osnatbm removed their assignment Jul 25, 2022
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.

Note that this also renames all build configuration variables that refer
to RocksDB from ROCKSDB_* to SPEEDB_* (e.g. ROCKSDB_LITE is now SPEEDB_LITE).
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes references in statuses as well as tools output.
isaac-io added a commit that referenced this issue Jul 25, 2022
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
@isaac-io isaac-io linked a pull request Jul 25, 2022 that will close this issue
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.

Note that this also renames all build configuration variables that refer
to RocksDB from ROCKSDB_* to SPEEDB_* (e.g. ROCKSDB_LITE is now SPEEDB_LITE).
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
isaac-io added a commit that referenced this issue Jul 25, 2022
This includes references in statuses as well as tools output.
isaac-io added a commit that referenced this issue Jul 25, 2022
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
@isaac-io isaac-io added the enhancement New feature or request label Jul 26, 2022
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.

Note that this also renames all build configuration variables that refer
to RocksDB from ROCKSDB_* to SPEEDB_* (e.g. ROCKSDB_LITE is now SPEEDB_LITE).
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes references in statuses as well as tools output.
isaac-io added a commit that referenced this issue Jul 27, 2022
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.

Note that this also renames all build configuration variables that refer
to RocksDB from ROCKSDB_* to SPEEDB_* (e.g. ROCKSDB_LITE is now SPEEDB_LITE).
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes references in statuses as well as tools output.
isaac-io added a commit that referenced this issue Jul 27, 2022
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.

Note that this also renames all build configuration variables that refer
to RocksDB from ROCKSDB_* to SPEEDB_* (e.g. ROCKSDB_LITE is now SPEEDB_LITE).
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
isaac-io added a commit that referenced this issue Jul 27, 2022
This includes references in statuses as well as tools output.
isaac-io added a commit that referenced this issue Jul 27, 2022
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
Yuval-Ariel pushed a commit that referenced this issue Apr 30, 2023
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.
Yuval-Ariel pushed a commit that referenced this issue Apr 30, 2023
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
Yuval-Ariel pushed a commit that referenced this issue Apr 30, 2023
This includes references in statuses as well as tools output.
Yuval-Ariel pushed a commit that referenced this issue Apr 30, 2023
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
Yuval-Ariel added a commit that referenced this issue May 2, 2023
which was accidentally removed in
general: replace RocksDB references in strings with Speedb (#64)
Yuval-Ariel pushed a commit that referenced this issue May 4, 2023
This updates the CMake and Makefile configuration and related scripts to
use variables where possible and avoid explicit references to RocksDB in
order to allow rebranding of the output artefacts.
Yuval-Ariel pushed a commit that referenced this issue May 4, 2023
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.
Yuval-Ariel pushed a commit that referenced this issue May 4, 2023
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
Yuval-Ariel pushed a commit that referenced this issue May 4, 2023
This includes references in statuses as well as tools output.
Yuval-Ariel pushed a commit that referenced this issue May 4, 2023
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
Yuval-Ariel added a commit that referenced this issue May 4, 2023
which was accidentally removed in
general: replace RocksDB references in strings with Speedb (#64)
udi-speedb pushed a commit that referenced this issue Oct 31, 2023
The change set includes:
1) A header file that contains the Major, Minor, and Patch versions of speedb
as a macro, as well as a set of functions returning info about how/when/where
this version of speedb was created.

also includes all changes done on build_version.cc.in in the following commits:
1. version: remove superfluous build property
The `speedb_build_spdb_key` property is unused and was accidentally
imported as part of #1.

2.general: replace RocksDB references in strings with Speedb (#64)
This includes references in statuses as well as tools output.
udi-speedb pushed a commit that referenced this issue Nov 13, 2023
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.
udi-speedb pushed a commit that referenced this issue Nov 13, 2023
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
udi-speedb pushed a commit that referenced this issue Nov 13, 2023
This includes references in statuses as well as tools output.
udi-speedb pushed a commit that referenced this issue Nov 13, 2023
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
udi-speedb pushed a commit that referenced this issue Nov 14, 2023
which was accidentally removed in
general: replace RocksDB references in strings with Speedb (#64)
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
This updates the CMake and Makefile configuration and related scripts to
use variables where possible and avoid explicit references to RocksDB in
order to allow rebranding of the output artefacts.
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
This includes references in statuses as well as tools output.
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
udi-speedb pushed a commit that referenced this issue Nov 15, 2023
which was accidentally removed in
general: replace RocksDB references in strings with Speedb (#64)
udi-speedb pushed a commit that referenced this issue Dec 1, 2023
The change set includes:
1) A header file that contains the Major, Minor, and Patch versions of speedb
as a macro, as well as a set of functions returning info about how/when/where
this version of speedb was created.

also includes all changes done on build_version.cc.in in the following commits:
1. version: remove superfluous build property
The `speedb_build_spdb_key` property is unused and was accidentally
imported as part of #1.

2.general: replace RocksDB references in strings with Speedb (#64)
This includes references in statuses as well as tools output.
udi-speedb pushed a commit that referenced this issue Dec 3, 2023
This updates the CMake and Makefile configuration and related scripts to
use variables where possible and avoid explicit references to RocksDB in
order to allow rebranding of the output artefacts.
udi-speedb pushed a commit that referenced this issue Dec 3, 2023
This includes both the CMake and Makefile configurations, various scripts
that are invoked during the build or during test runs, as well as the
Java native library loader code.
udi-speedb pushed a commit that referenced this issue Dec 3, 2023
This includes simple renaming from `rocksdb` to `speedb` as well as replacing
URLs that are pointing to the RocksDB repository with the URL of the Speedb
repository.
udi-speedb pushed a commit that referenced this issue Dec 3, 2023
This includes references in statuses as well as tools output.
udi-speedb pushed a commit that referenced this issue Dec 3, 2023
This is done across all tools except trace_replay, which specifically
looks for the RocksDB version for database compatibility checks.
udi-speedb pushed a commit that referenced this issue Dec 5, 2023
which was accidentally removed in
general: replace RocksDB references in strings with Speedb (#64)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants