You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@crail.apache.org by GitBox <gi...@apache.org> on 2021/02/15 19:47:00 UTC

[GitHub] [incubator-crail] mbrodmann commented on a change in pull request #84: Resource Elasticity [1/3]

mbrodmann commented on a change in pull request #84:
URL: https://github.com/apache/incubator-crail/pull/84#discussion_r576394716



##########
File path: client/src/main/java/org/apache/crail/metadata/DataNodeStatistics.java
##########
@@ -22,25 +22,29 @@
 import java.nio.ByteBuffer;
 
 public class DataNodeStatistics {
-	public static final int CSIZE = 12;
+	public static final int CSIZE = 14;
 	
 	private long serviceId;
 	private int freeBlockCount;
+	private short status;

Review comment:
       Yes, that is correct. Right now the status field is only changed to RpcErrors.ERR_DATANODE_STOP when a datanode is selected for shutdown. Replacing it with a new class seems like a good idea.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org