#3499: Backport of new ZSTD compression
------------------------------+-------------------------
Reporter: neteler | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.4.1
Component: LibRaster | Version: unspecified
Keywords: ZSTD compression | CPU: All
Platform: All |
------------------------------+-------------------------
Carried over from #2750
Replying to [#2750 comment:69 neteler]:
> Concerning ZSTD compression: backport candidate for 7.4.1.
... for this I identified
* r71892 update documentation for GRASS compression methods
* r71891 libraster: update for ZSTD, use new G_compress_bound()
* r71890 libgis: add ZSTD (Zstandard) compression
* r71889 add ZSTD (Zstandard) compression to the Make system
Probably also helpful:
* r72031 libgis: more informative messages when (de)compression fails
* r72042 libgis: add warnings when writing compressed data fails
#3499: Backport of new ZSTD compression
--------------------------+------------------------------
Reporter: neteler | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.4.1
Component: LibRaster | Version: unspecified
Resolution: | Keywords: ZSTD compression
CPU: All | Platform: All
--------------------------+------------------------------
Comment (by mmetz):
Replying to [ticket:3499 neteler]:
> Carried over from #2750
>
> Replying to [#2750 comment:69 neteler]:
> > Concerning ZSTD compression: backport candidate for 7.4.1.
IMHO ZSTD compression is not a backport candidate because it is a new
feature. Moreover, substantial changes were introduced to both the gis and
raster library, which are in no way a bugfix, disqualifying it for
backporting.
#3499: Backport of new ZSTD compression
--------------------------+------------------------------
Reporter: neteler | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.4.1
Component: LibRaster | Version: unspecified
Resolution: | Keywords: ZSTD compression
CPU: All | Platform: All
--------------------------+------------------------------
Description changed by neteler:
Old description:
Carried over from #2750
Replying to [#2750 comment:69 neteler]:
> Concerning ZSTD compression: backport candidate for 7.4.1.
... for this I identified
* r71892 update documentation for GRASS compression methods
* r71891 libraster: update for ZSTD, use new G_compress_bound()
* r71890 libgis: add ZSTD (Zstandard) compression
* r71889 add ZSTD (Zstandard) compression to the Make system
Probably also helpful:
* r72031 libgis: more informative messages when (de)compression fails
* r72042 libgis: add warnings when writing compressed data fails
Is anything missing from this list?
New description:
Carried over from #2750
Replying to [#2750 comment:69 neteler]:
> Concerning ZSTD compression: backport candidate for 7.4.1.
... for this I identified
* r71892 update documentation for GRASS compression methods
* r71891 libraster: update for ZSTD, use new G_compress_bound()
* r71890 libgis: add ZSTD (Zstandard) compression
* r71889 add ZSTD (Zstandard) compression to the Make system
Probably also helpful:
* r72030 libraster: more informative messages when reading a row fails
* r72031 libgis: more informative messages when (de)compression fails
* r72042 libgis: add warnings when writing compressed data fails
#3499: Backport of new ZSTD compression
--------------------------+------------------------------
Reporter: neteler | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone: 7.4.1
Component: LibRaster | Version: unspecified
Resolution: | Keywords: ZSTD compression
CPU: All | Platform: All
--------------------------+------------------------------
Comment (by mmetz):
Replying to [comment:1 mmetz]:
> Replying to [ticket:3499 neteler]:
> > Carried over from #2750
> >
> > Replying to [#2750 comment:69 neteler]:
> > > Concerning ZSTD compression: backport candidate for 7.4.1.
>
> IMHO ZSTD compression is not a backport candidate because it is a new
feature. Moreover, substantial changes were introduced to both the gis and
raster library, which are in no way a bugfix, disqualifying it for
backporting.
Emphasizing the backporting issue: backporting (in this case) is not
trivial because it involves several low-level changes. If you want to use
ZSTD compression, use trunk r71892. If you want more meaningful messages
about why (de-)compression fails, use trunk r72042.
Equivalent for any other new features in trunk: use trunk with the
revision when this new feature was added.
Feedback to new features helps to keep trunk robust (suitable for everyday
work).