Skip to content

Merge develop.jpeg2000 with the latest develop#89

Open
nalinigans wants to merge 35 commits into
develop.jpeg2000from
ng_jpeg2k
Open

Merge develop.jpeg2000 with the latest develop#89
nalinigans wants to merge 35 commits into
develop.jpeg2000from
ng_jpeg2k

Conversation

@nalinigans

@nalinigans nalinigans commented Feb 19, 2021

Copy link
Copy Markdown
Contributor

This gets all the changes merged in, but have pushed jpeg2k validation changes out of tiledb.cc and into codec.[h|cc] for now. Once, we have a codec registering scheme in place all these changes can migrate with codec_jpeg.[h|cc] where the codec is implemented.

@kgururaj, do you mind if the blosc support is pulled out? Also, maybe it will be a good time to move array_schema to a json format when we implement a plugin sort of a mechanism for codecs.

nalinigans and others added 30 commits February 20, 2020 14:27
…ing a cached instance as there are multithreading issues that need to be figured out (#52)
* Add hdfs multithreading unit tests
* Change create_workspace to adhere to the contract in initialize_workspace for consistency
…ssion (#54)

Add array_schema tests before changing code to support compression for offsets.

Also, fix broken GCS builds -
   Move to HDFS 2.9.2 for compatibility with latest GCS Connector
   Replace GCS deprecated properties.
…the compression of the attributes themselves. (#55)

* Support compression separately from the attribute for the offsets to attributes with variable number of cells
* Factor in the search tile used when reading

* Clean up handling of offsets in array schema

* Cleanup test_sparse_array_api as the compatibility test is at the test_array_schema_api level

* Address review comments

* Address review comments
…LE_HANDLES_OPEN is set. Also, set up benchmark harness. (#56)

* Allow for file handles to also stay open until close() when TILEDB_DISABLE_FILE_LOCKING=1

* Add benchmarking harness and support TILEDB_KEEP_FILE_HANDLES_OPEN env instead of piggy backing this option through TILEDB_DISABLE_FILE_LOCKS

* Cleanup compiler warnings
…ing. (#57)

* Support Delta Encoding as a precompression filter
* Adding test utility to print out TileDB array schema
* gcc requires string.h include for memset
…ry (#59)

* Add lz4 and bitshuffle from the kiyo-masui/bitshuffle GitHub repository
…s. (#61)

* Support BitShuffle as a pre compression filter so it can be combined with other codecs rather than just lz4

* Allow for an external LZ4 library. This will allow for GenomicsDB to use the Intel IPP Optimized LZ4 library

* Fix travis builds as lz4 codec is included in the repository

* Move lz4 sources to the latest 1.9.2 released version

* Add error checking with bshuf_bitshuffle specific error codes

* Compute offset compression levels separate from compression levels while creating Codec
…ontext. (#63)

* Add support for enable_shared_posixfs_optimizations through tiledb context. This sets disable_file_locking and keep_file_handles_open for posix filesystems

* Allow TileDBUtils methods to set enable_shared_posixfs_optimizations options via tiledb context

* Just delete the workspace dir When replace is true for intialize_workspace as they may have non-tiledb artifacts (e.g. GenomicsDB workspaces)
* Allow read-only locking when arrays are opened in read mode

* Change to match changes in consolidation locking behavior

Co-authored-by: nalinigans <nalini@omicsautomation.com>
* Move build to xenial
* Fix reference to JAVA_HOME in the travis resources
* Initial direct Azure Blob Storage support using C++ Lite client
* Fix doc and rename variable to rationalize azure::download_blob_to_stream usage

* First pass at support for buffered IO

* Implement chunked buffering for reading/writing tiles

* Use size_t/off_t for all file read/write buffer methods

* Refine the buffered read algorithm
* Implement github action workflows for hdfs/gcs/azure builds
* Use trailing underscore with class variables
* Cleanup run_dfs_tests script
…79)

* Rename url references to uri

* Remove references to master catalog and tiledb_home_ as this is not used anymore

* Move common cloud helper routines to base class storage_fs

* Update copyrights
nalinigans and others added 5 commits January 17, 2021 21:31
…) to return ssize_t (#82)

* Fix deletion of new array objects in benchmark test

* Adding checks for all malloc/realloc calls and refactoring file_size() to return ssize_t to allow for return values of -1

* Copyright updates and adding to tests
* Include support for azure blob endpoints
* Install azurite to run azure tests in GitHub workflow
…ing with github actions. (#85)

* cmake support for building with aws sdk

* add_dependencies to tiledb objects for resolving include dependencies to awssdk

* awssdk core modules seem to require the macos framework while creating lbtiledb.dylib, does not require the framework for the static library though.

* Fix what looks like a transitive dependency with the framework on Mac with awssdk static libraries - not sure why they introduce this
* Support S3 using aws-sdk-cpp libraries. Also setup minio for testing this with github actions.

* Allow github actions to test on aws and minio

*  Support multiple builds running concurrently with run_dfs_tests
@nalinigans nalinigans requested a review from KLMatlock February 19, 2021 00:41
@codecov-io

codecov-io commented Feb 19, 2021

Copy link
Copy Markdown

Codecov Report

Merging #89 (0fe0c26) into develop.jpeg2000 (5d69a47) will increase coverage by 0.01%.
The diff coverage is 78.64%.

Impacted file tree graph

@@                 Coverage Diff                  @@
##           develop.jpeg2000      #89      +/-   ##
====================================================
+ Coverage             61.67%   61.69%   +0.01%     
====================================================
  Files                    44       56      +12     
  Lines                 16205    17390    +1185     
====================================================
+ Hits                   9994    10728     +734     
- Misses                 6211     6662     +451     
Impacted Files Coverage Δ
core/include/codec/codec_blosc.h 77.27% <ø> (ø)
core/include/codec/codec_gzip.h 100.00% <ø> (ø)
core/include/codec/codec_rle.h 0.00% <ø> (ø)
core/include/misc/utils.h 100.00% <ø> (ø)
core/include/storage_manager/storage_manager.h 100.00% <ø> (ø)
core/src/codec/codec_rle.cc 1.29% <0.00%> (ø)
core/src/misc/hilbert_curve.cc 1.35% <0.00%> (-0.02%) ⬇️
core/src/storage_manager/storage_gcs.cc 1.61% <0.00%> (-0.03%) ⬇️
core/src/array/array.cc 54.75% <40.00%> (-0.85%) ⬇️
core/src/storage_manager/storage_manager_config.cc 63.01% <47.82%> (-9.87%) ⬇️
... and 70 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d69a47...0fe0c26. Read the comment docs.

@nalinigans nalinigans changed the base branch from develop.jpeg2000 to develop February 24, 2021 21:21
@nalinigans nalinigans changed the base branch from develop to develop.jpeg2000 February 24, 2021 21:22
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.

4 participants