[
https://issues.apache.org/jira/browse/GEODE-3022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16278924#comment-16278924
]
ASF GitHub Bot commented on GEODE-3022:
---------------------------------------
mmartell closed pull request #162: GEODE-3022: Removing a duplicate test
URL: https://github.com/apache/geode-native/pull/162
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/cppcache/integration-test/testSystemProperties.cpp
b/cppcache/integration-test/testSystemProperties.cpp
index f882a0bb..11ada78b 100644
--- a/cppcache/integration-test/testSystemProperties.cpp
+++ b/cppcache/integration-test/testSystemProperties.cpp
@@ -58,37 +58,6 @@ BEGIN_TEST(DEFAULT)
}
END_TEST(DEFAULT)
-/*
- * Commenting the last two tests because I need to know where to put
- * the file which will be passed to the constructor. These two tests works.
- * But for others it might fail so commenting them as of now.
- */
-BEGIN_TEST(CONFIG)
- {
- // create a file for alternate properties...
- // FILE* propFile = ACE_OS::fopen( "./test.properties", "a+" );
- // ACE_OS::fprintf( propFile, "gf.transport.config=./gfconfig\n" );
- /// ACE_OS::fprintf( propFile, "statistics.sample.interval=2000\n" );
- // ACE_OS::fprintf( propFile, "statistics.enabled=false\n" );
- // ACE_OS::fprintf( propFile, "statistics.archive.file=./stats.gfs\n" );
- // ACE_OS::fprintf( propFile, "log.level=error\n" );
- // ACE_OS::fclose( propFile );
-
- // Make sure product can at least log to stdout.
- Log::init(Log::Config, nullptr, 0);
-
- SystemProperties* sp = new SystemProperties(nullptr, "test.properties");
- ASSERT(sp->statisticsSampleInterval() == std::chrono::seconds(1),
- "expected 1");
- ASSERT(sp->statisticsEnabled() == true, "expected true");
- auto&& safname = sp->statisticsArchiveFile();
- ASSERT_EQ("statArchive.gfs", safname);
- Log::LogLevel ll = sp->logLevel();
- ASSERT(ll == Log::Config, "expected Log::Config");
- delete sp;
- }
-END_TEST(CONFIG)
-
BEGIN_TEST(NEW_CONFIG)
{
// When the tests are run from the build script the environment variable
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> [JGRP00001] configuration error: the following properties in
> org.apache.geode.distributed.internal.membership.gms.messenger.Transport are
> not recognized: {ignore_dont_bundle=false}
> ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: GEODE-3022
> URL: https://issues.apache.org/jira/browse/GEODE-3022
> Project: Geode
> Issue Type: Bug
> Reporter: Sumitra Chatterjee
> Priority: Blocker
>
> Getting above exception when trying to start cache server. Further checked in
> JGroupsMessenger.java and found below:
> void setMessageFlags(DistributionMessage gfmsg, Message msg) {
> // Bundling is mostly only useful if we're doing no-ack work,
> // which is fairly rare
> msg.setFlag(Flag.DONT_BUNDLE);
> As per https://issues.jboss.org/browse/JGRP-1737, DONT_BUNDLE is probably not
> to be used (correct me if I am misunderstanding). Any ideas how to get the
> server start up would be very helpful.
> Thanks
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
