-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26140/
-----------------------------------------------------------

Review request for Flume.


Bugs: FLUME-2475
    https://issues.apache.org/jira/browse/FLUME-2475


Repository: flume-git


Description
-------

Use Locale.ENGLISH for every toUpperCase/toLowerCase call. This prevents bugs 
when the default system locale is not en_US.


Diffs
-----

  
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/encryption/CipherProviderFactory.java
 d5e9353 
  
flume-ng-channels/flume-file-channel/src/main/java/org/apache/flume/channel/file/encryption/KeyProviderFactory.java
 60d51ed 
  
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/DatabaseType.java
 e26cb84 
  
flume-ng-channels/flume-jdbc-channel/src/main/java/org/apache/flume/channel/jdbc/TransactionIsolation.java
 ed157cf 
  
flume-ng-channels/flume-jdbc-channel/src/test/java/org/apache/flume/channel/jdbc/TestDatabaseTypeEnum.java
 55ad1eb 
  
flume-ng-channels/flume-jdbc-channel/src/test/java/org/apache/flume/channel/jdbc/TestTransactionIsolationLevelEnum.java
 ea0bac3 
  
flume-ng-clients/flume-ng-log4jappender/src/main/java/org/apache/flume/clients/log4jappender/Log4jAvroHeaders.java
 08a7203 
  
flume-ng-configuration/src/main/java/org/apache/flume/conf/ComponentConfigurationFactory.java
 c867a03 
  
flume-ng-configuration/src/main/java/org/apache/flume/conf/FlumeConfiguration.java
 f214149 
  
flume-ng-configuration/src/main/java/org/apache/flume/conf/source/SourceConfiguration.java
 13c3360 
  
flume-ng-core/src/main/java/org/apache/flume/channel/ChannelSelectorFactory.java
 998c8c0 
  
flume-ng-core/src/main/java/org/apache/flume/channel/DefaultChannelFactory.java 
06e1763 
  
flume-ng-core/src/main/java/org/apache/flume/client/avro/ReliableSpoolingFileEventReader.java
 0bc3f23 
  
flume-ng-core/src/main/java/org/apache/flume/instrumentation/MonitoredCounterGroup.java
 1d3f0f1 
  
flume-ng-core/src/main/java/org/apache/flume/interceptor/InterceptorBuilderFactory.java
 50e6fbc 
  
flume-ng-core/src/main/java/org/apache/flume/serialization/AvroEventDeserializer.java
 e44978f 
  
flume-ng-core/src/main/java/org/apache/flume/serialization/EventDeserializerFactory.java
 80b9cc3 
  
flume-ng-core/src/main/java/org/apache/flume/serialization/EventSerializerFactory.java
 dbb3935 
  
flume-ng-core/src/main/java/org/apache/flume/serialization/LineDeserializer.java
 d8e5ae2 
  flume-ng-core/src/main/java/org/apache/flume/sink/DefaultSinkFactory.java 
0e49e0b 
  flume-ng-core/src/main/java/org/apache/flume/sink/SinkProcessorFactory.java 
1499872 
  flume-ng-core/src/main/java/org/apache/flume/source/DefaultSourceFactory.java 
95cf13d 
  flume-ng-core/src/main/java/org/apache/flume/source/SpoolDirectorySource.java 
d0c2e99 
  flume-ng-core/src/main/java/org/apache/flume/source/SyslogUtils.java 208fefe 
  flume-ng-core/src/main/java/org/apache/flume/tools/DirectMemoryUtils.java 
b12571b 
  flume-ng-core/src/main/java/org/apache/flume/tools/PlatformDetect.java 
3beec06 
  
flume-ng-embedded-agent/src/main/java/org/apache/flume/agent/embedded/EmbeddedAgentConfiguration.java
 06ea7c0 
  flume-ng-node/src/main/java/org/apache/flume/node/Application.java 832285a 
  flume-ng-sdk/src/main/java/org/apache/flume/api/RpcClientFactory.java 00d9cf0 
  
flume-ng-sinks/flume-dataset-sink/src/main/java/org/apache/flume/sink/kite/DatasetSink.java
 4cd3027 
  
flume-ng-sinks/flume-hdfs-sink/src/main/java/org/apache/flume/sink/hdfs/AvroEventSerializer.java
 4b8fc78 
  
flume-ng-sources/flume-jms-source/src/main/java/org/apache/flume/source/jms/JMSSource.java
 addd97a 
  flume-tools/src/main/java/org/apache/flume/tools/FlumeToolType.java f886c89 

Diff: https://reviews.apache.org/r/26140/diff/


Testing
-------

Passes tests correctly.


Thanks,

Santiago Mola

Reply via email to