Skip to content

Commit

Permalink
Mac compatability
Browse files Browse the repository at this point in the history
  • Loading branch information
EthanSteinberg committed Aug 7, 2024
1 parent 5e6b391 commit bd80a14
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/python-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,7 @@ jobs:
Publish Python 🐍 distribution 📦 to PyPI
if: startsWith(github.ref, 'refs/tags/') # only publish to PyPI on tag pushes
needs:
- build_wheels
- make_sdist
- make_artifact
runs-on: ubuntu-latest
environment:
name: pypi
Expand Down
5 changes: 2 additions & 3 deletions native/perform_etl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -194,13 +194,12 @@ std::vector<std::string> get_samples(
moodycamel::BlockingConcurrentQueue<absl::optional<std::string>>&
file_queue) {
arrow::MemoryPool* pool = arrow::default_memory_pool();
absl::flat_hash_map<uint64_t, uint8_t> string_status;
absl::flat_hash_map<size_t, uint8_t> string_status;

std::vector<std::string> unique_strings;

auto process_function = [&](std::string_view item) {
static_assert(std::is_same_v<uint64_t, size_t>);
uint64_t h = std::hash<decltype(item)>{}(item);
size_t h = std::hash<decltype(item)>{}(item);
uint8_t& value = string_status[h];

if (value == 0) {
Expand Down

0 comments on commit bd80a14

Please sign in to comment.