iGN5117 commented on code in PR #1014:
URL: https://github.com/apache/sedona/pull/1014#discussion_r1323176900
##########
common/src/main/java/org/apache/sedona/common/Constants/RasterConstants.java:
##########
@@ -0,0 +1,6 @@
+package org.apache.sedona.common.Constants;
+
+public class RasterConstants {
+
+ public static final String INVALID_BAND = "Provided band index %d is not
present in the raster";
Review Comment:
I believe it is necessary to keep these files isolated. One very important
reason is that down the line, we're eventually going to start supporting
translation into other languages like postgis does.
Also, it is very much easier to change error messages etc whenever needed.
At both of these times, having a separate file is the standard practice
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
