After pulling new trunk and compiling, thrift generated files show changes in 'svn status'

[David Medinets <da...@gmail.com>]

I pulled trunk into a new directory. Then ran 'mvn package -P
assemble'. Then ran 'svn status'. Normally I see no changed files.
However, this time I saw the following:

$ svn status
M       core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ClientService.java
M       core/src/main/java/org/apache/accumulo/core/master/thrift/MasterClientService.java
M       core/src/main/java/org/apache/accumulo/core/master/thrift/MasterMonitorInfo.java
M       core/src/main/java/org/apache/accumulo/core/master/thrift/TabletServerStatus.java
M       core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java
M       core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TabletClientService.java
M       core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ActiveScan.java
M       core/src/main/java/org/apache/accumulo/core/data/thrift/MultiScanResult.java
M       core/src/main/java/org/apache/accumulo/core/data/thrift/UpdateErrors.java

I assume these are generated files? Here is a sample of the differences;

medined@affy:~/workspace/accumulo_trunk$ svn diff
core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java
Index: core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java
===================================================================
--- core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java
      (revision 1365779)
+++ core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java
      (working copy)
@@ -601,7 +601,7 @@
                 for (int _i45 = 0; _i45 < _map44.size; ++_i45)
                 {
                   String _key46; // required
-                  String _val47; // optional
+                  String _val47; // required
                   _key46 = iprot.readString();
                   _val47 = iprot.readString();
                   struct.properties.put(_key46, _val47);
@@ -729,7 +729,7 @@
           for (int _i51 = 0; _i51 < _map50.size; ++_i51)
           {
             String _key52; // required
-            String _val53; // optional
+            String _val53; // required
             _key52 = iprot.readString();
             _val53 = iprot.readString();
             struct.properties.put(_key52, _val53);

Should these files be ignored? Checked back in? Recommendations?