332 comsngularkloadgenserializerprotobufserializer nullpointerexception #430

jemacineiras · 2024-01-08T23:22:00Z

No description provided.

Added a trim statement to avoid problem when cast the schemas from classpath

Removed some unused test files

…rializer-nullpointerexception

Apparently the original project was renamed into cmak to avoid break the TOS of kafka having the kafka in the name of unofficial tool

…genserializerprotobufserializer-nullpointerexception # Conflicts: # pom.xml # src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java # src/test/resources/mappings/schema_registry_stub.json

…rializer-nullpointerexception

…pointerexception' of github.com:sngular/kloadgen into 332-comsngularkloadgenserializerprotobufserializer-nullpointerexception

…rializer-nullpointerexception # Conflicts: # pom.xml # src/main/java/com/sngular/kloadgen/config/schemaregistry/SchemaRegistryConfigElementBeanInfo.java # src/main/java/com/sngular/kloadgen/extractor/extractors/ExtractorFactory.java # src/main/java/com/sngular/kloadgen/extractor/extractors/avro/AvroExtractor.java # src/main/java/com/sngular/kloadgen/sampler/KafkaProducerSampler.java # src/test/java/com/sngular/kloadgen/sampler/SamplerUtilTest.java

…rializer-nullpointerexception

github-actions · 2024-01-08T23:22:16Z

Project version has not been updated in pom.xml. Please, update your version using https://semver.org specifications.

github-actions · 2024-01-08T23:26:17Z

Project version has not been updated in pom.xml. Please, update your version using https://semver.org specifications.

…rializer-nullpointerexception

janDuinRod · 2024-05-02T15:44:11Z

src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java

Maybe it would be a good idea to delete all this commented out code? If it is necessary in the future we could just get it from version control.

janDuinRod · 2024-05-02T15:48:37Z

src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java

+    if (Objects.nonNull(context.getJMeterVariables().get(PropsKeysHelper.KEY_DESERIALIZER_CLASS_PROPERTY))) {
+      props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, context.getJMeterVariables().get(PropsKeysHelper.KEY_DESERIALIZER_CLASS_PROPERTY));
    } else {
      props.put(ConsumerConfig.KEY_DESERIALIZER_CLASS_CONFIG, "org.apache.kafka.common.serialization.StringDeserializer");
    }
-    if (Objects.nonNull(JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.VALUE_DESERIALIZER_CLASS_PROPERTY))) {
-      props.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.VALUE_DESERIALIZER_CLASS_PROPERTY));
+    if (Objects.nonNull(context.getJMeterVariables().get(PropsKeysHelper.VALUE_DESERIALIZER_CLASS_PROPERTY))) {
+      props.put(ConsumerConfig.VALUE_DESERIALIZER_CLASS_CONFIG, context.getJMeterVariables().get(PropsKeysHelper.VALUE_DESERIALIZER_CLASS_PROPERTY));


Maybe its a good idea to add a vars variable holding the content of context.getJMeterVariables() as we do in other methods in order to reduce the number of times we repeat this expression.

janDuinRod · 2024-05-02T15:49:05Z

src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java

+    if (Objects.nonNull(context.getJMeterVariables().get(ProducerKeysHelper.VALUE_NAME_STRATEGY))) {
+      props.put(ProducerKeysHelper.VALUE_NAME_STRATEGY, context.getJMeterVariables().get(ProducerKeysHelper.VALUE_NAME_STRATEGY));
    }
-    if (Objects.nonNull(JavaSamplerContext.getJMeterVariables().get(ProducerKeysHelper.KEY_NAME_STRATEGY))) {
-      props.put(ProducerKeysHelper.KEY_NAME_STRATEGY, JavaSamplerContext.getJMeterVariables().get(ProducerKeysHelper.KEY_NAME_STRATEGY));
+    if (Objects.nonNull(context.getJMeterVariables().get(ProducerKeysHelper.KEY_NAME_STRATEGY))) {
+      props.put(ProducerKeysHelper.KEY_NAME_STRATEGY, context.getJMeterVariables().get(ProducerKeysHelper.KEY_NAME_STRATEGY));


Same thing as the comment before

janDuinRod · 2024-05-02T15:49:34Z

src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java

-    if (Objects.nonNull(JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.VALUE_SCHEMA))) {
-      props.put(PropsKeysHelper.VALUE_SCHEMA, JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.VALUE_SCHEMA));
+    if (Objects.nonNull(context.getJMeterVariables().get(PropsKeysHelper.VALUE_SCHEMA))) {
+      props.put(PropsKeysHelper.VALUE_SCHEMA, context.getJMeterVariables().get(PropsKeysHelper.VALUE_SCHEMA));
    }
-    if (Objects.nonNull(JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.KEY_SCHEMA))) {
-      props.put(PropsKeysHelper.KEY_SCHEMA, JavaSamplerContext.getJMeterVariables().get(PropsKeysHelper.KEY_SCHEMA));
+    if (Objects.nonNull(context.getJMeterVariables().get(PropsKeysHelper.KEY_SCHEMA))) {
+      props.put(PropsKeysHelper.KEY_SCHEMA, context.getJMeterVariables().get(PropsKeysHelper.KEY_SCHEMA));
    }
-    if (Objects.nonNull(JavaSamplerContext.getJMeterVariables().get(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL))) {
-      props.put(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL, JavaSamplerContext.getJMeterVariables().get(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL));
+    if (Objects.nonNull(context.getJMeterVariables().get(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL))) {
+      props.put(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL, context.getJMeterVariables().get(SchemaRegistryKeyHelper.SCHEMA_REGISTRY_URL));


Same thing as the comment before

AdrianLagartera and others added 30 commits February 22, 2023 13:45

#332 Add Files Modified To Solve Exception

b21e311

Added a trim statement to avoid problem when cast the schemas from classpath

#332 Add get dependencies and build proto.

9a197c7

#332 Add Test And Documentation

af0dfe3

Removed some unused test files

#332 Add Topic Name Strategy

a129cd7

Merge branch 'master' into 332-comsngularkloadgenserializerprotobufse…

3f5d91c

…rializer-nullpointerexception

#332 Change image from kafka manager to newest version

b8426a9

Apparently the original project was renamed into cmak to avoid break the TOS of kafka having the kafka in the name of unofficial tool

Get the schema name from the schema references of imported classes

4858372

#332 Removed generator set up by TopicNameStrategy

0fd3920

#332 refactor

dd531d8

Merge remote-tracking branch 'origin/master' into 332-comsngularkload…

43ed7da

…genserializerprotobufserializer-nullpointerexception # Conflicts: # pom.xml # src/main/java/com/sngular/kloadgen/sampler/SamplerUtil.java # src/test/resources/mappings/schema_registry_stub.json

#332 refactor

b037bdf

#332 refactor

82a445c

#332 updated pom version

622398a

#332 fixes PR

eb5f8b9

#332 fixes PR

e86011b

update pom

5fa0649

Merge with Master

f2656b8

Merge with Master

1994dff

#340 update dependencies

88978ce

#332 Adding google proto types

46e047b

#332 Fixing size serializer.

0324595

#332 Include proto files in package

d15d7b2

Merge branch 'master' into 332-comsngularkloadgenserializerprotobufse…

20f07ba

…rializer-nullpointerexception

#332 Fix styles and pom version

a3cad52

Merge branch '332-comsngularkloadgenserializerprotobufserializer-null…

a4440bc

…pointerexception' of github.com:sngular/kloadgen into 332-comsngularkloadgenserializerprotobufserializer-nullpointerexception

#332 Fix styles and pom version

a442f7a

#332 Fix styles and test

8b8e936

#332 Fix styles

6a3d7e7

#332 Try to process multi level protobuff dependencies

b10e556

#332 Try to process multi level protobuff dependencies

dced5f8

jemacineiras added 6 commits November 13, 2023 22:46

#332 Fix Google dependencies

79b8cc2

#332 Fix generator calculation

7929bfe

#332 Fix generator calculation

da2c94b

#332 Merge with master.

669fd62

Merge branch 'master' into 332-comsngularkloadgenserializerprotobufse…

bed88da

…rializer-nullpointerexception

jemacineiras added bug Something isn't working release Adding this tag to a PR will cause a release on merge labels Jan 8, 2024

jemacineiras requested review from RobertoSngular, GraciMndzSNG and ramilsngular January 8, 2024 23:22

jemacineiras self-assigned this Jan 8, 2024

jemacineiras and others added 12 commits January 9, 2024 17:04

#332 Update POM.

e9cefa8

#332 Fix Protobuf Apicurio Extractor.

00250b9

#332 fix new version static error

15a30ed

#332 fix new version static error

c8de4d0

#332 fix Generator creation

25b7a0d

#332 fix Random Object generation

ba55fd6

#332 fix Serializer generation

addc4df

#332 fix generator creation

a94d288

Merge branch 'master' into 332-comsngularkloadgenserializerprotobufse…

d147850

…rializer-nullpointerexception

#332 add test

65a724d

Update pom.xml version and developers

24053c2

Update pom-maven-central.xml

153c178

janDuinRod reviewed May 2, 2024

View reviewed changes

davidgayoso approved these changes May 3, 2024

View reviewed changes

davidgayoso merged commit 4540f78 into master May 3, 2024
7 checks passed

janDuinRod deleted the 332-comsngularkloadgenserializerprotobufserializer-nullpointerexception branch May 3, 2024 09:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

332 comsngularkloadgenserializerprotobufserializer nullpointerexception #430

332 comsngularkloadgenserializerprotobufserializer nullpointerexception #430

jemacineiras commented Jan 8, 2024

github-actions bot commented Jan 8, 2024

github-actions bot commented Jan 8, 2024

janDuinRod May 2, 2024

janDuinRod May 2, 2024

janDuinRod May 2, 2024

janDuinRod May 2, 2024

332 comsngularkloadgenserializerprotobufserializer nullpointerexception #430

332 comsngularkloadgenserializerprotobufserializer nullpointerexception #430

Conversation

jemacineiras commented Jan 8, 2024

github-actions bot commented Jan 8, 2024

github-actions bot commented Jan 8, 2024

janDuinRod May 2, 2024

Choose a reason for hiding this comment

janDuinRod May 2, 2024

Choose a reason for hiding this comment

janDuinRod May 2, 2024

Choose a reason for hiding this comment

janDuinRod May 2, 2024

Choose a reason for hiding this comment